Merge pull request #6969 from phivh/fix/blank-screen-when-user-denies…

…-camera-access fix: App shows blank screen when user denies camera access
Expensify · Jan 6, 2022 · 5863d0f · 5863d0f
2 parents 51de5d9 + 2f5603d
commit 5863d0f
Show file tree

Hide file tree

Showing 4 changed files with 39 additions and 2 deletions.
diff --git a/src/CONST.js b/src/CONST.js
@@ -428,6 +428,10 @@ const CONST = {
         ERROR: {
             USER_CANCELLED: 'User canceled flow',
             USER_TAPPED_BACK: 'User exited by clicking the back button.',
+            USER_CAMERA_DENINED: 'Onfido.OnfidoFlowError',
+            USER_CAMERA_PERMISSION: 'Encountered an error: cameraPermission',
+            // eslint-disable-next-line max-len
+            USER_CAMERA_CONSENT_DENIED: 'Unexpected result Intent. It might be a result of incorrect integration, make sure you only pass Onfido intent to handleActivityResult. It might be due to unpredictable crash or error. Please report the problem to [email protected]. Intent: null \n resultCode: 0',
         },
     },
 

diff --git a/src/components/Onfido/index.native.js b/src/components/Onfido/index.native.js
@@ -1,6 +1,7 @@
 import _ from 'underscore';
 import lodashGet from 'lodash/get';
 import React from 'react';
+import {Alert, Linking} from 'react-native';
 import {
     Onfido as OnfidoSDK,
     OnfidoCaptureType,
@@ -45,6 +46,36 @@ class Onfido extends React.Component {
                     return;
                 }
 
+                // Handle user camera permission on iOS and Android
+                if (_.contains(
+                    [
+                        CONST.ONFIDO.ERROR.USER_CAMERA_PERMISSION,
+                        CONST.ONFIDO.ERROR.USER_CAMERA_DENINED,
+                        CONST.ONFIDO.ERROR.USER_CAMERA_CONSENT_DENIED,
+                    ],
+                    errorMessage,
+                )) {
+                    Alert.alert(
+                        this.props.translate('onfidoStep.cameraPermissionsNotGranted'),
+                        this.props.translate('onfidoStep.cameraRequestMessage'),
+                        [
+                            {
+                                text: this.props.translate('common.cancel'),
+                                onPress: () => this.props.onUserExit(),
+                            },
+                            {
+                                text: this.props.translate('common.settings'),
+                                onPress: () => {
+                                    this.props.onUserExit();
+                                    Linking.openSettings();
+                                },
+                            },
+                        ],
+                        {cancelable: false},
+                    );
+                    return;
+                }
+
                 this.props.onError(errorMessage);
             });
     }

diff --git a/src/languages/en.js b/src/languages/en.js
@@ -512,7 +512,6 @@ export default {
         selfSelect: 'Self-select',
         callMeByMyName: 'Call me by my name',
     },
-    cameraPermissionsNotGranted: 'Camera permissions not granted',
     messages: {
         errorMessageInvalidPhone: 'Please enter a valid phone number without brackets or dashes. If you\'re outside the US please include your country code, eg. +447782339811',
         maxParticipantsReached: 'You\'ve reached the maximum number of participants for a group chat.',
@@ -523,6 +522,8 @@ export default {
         tryAgain: 'Try again',
         verifyIdentity: 'Verify identity',
         genericError: 'There was an error while processing this step. Please try again.',
+        cameraPermissionsNotGranted: 'Camera permissions not granted',
+        cameraRequestMessage: 'You have not granted us camera access. We need access to complete verification.',
     },
     additionalDetailsStep: {
         headerTitle: 'Additional details',

diff --git a/src/languages/es.js b/src/languages/es.js
@@ -512,7 +512,6 @@ export default {
         selfSelect: 'Personalízalo',
         callMeByMyName: 'Llámame por mi nombre',
     },
-    cameraPermissionsNotGranted: 'No has habilitado los permisos para acceder a la cámara',
     messages: {
         errorMessageInvalidPhone: 'Por favor, introduce un número de teléfono válido sin paréntesis o guiones. Si reside fuera de Estados Unidos, por favor incluye el prefijo internacional. P. ej. +447782339811',
         maxParticipantsReached: 'Has llegado al número máximo de participantes para un grupo.',
@@ -523,6 +522,8 @@ export default {
         tryAgain: 'Intentar otra vez',
         verifyIdentity: 'Verificar identidad',
         genericError: 'Hubo un error al procesar este paso. Inténtalo de nuevo.',
+        cameraPermissionsNotGranted: 'No has habilitado los permisos para acceder a la cámara',
+        cameraRequestMessage: 'No has habilitado los permisos para acceder a la cámara. Necesitamos acceso para completar la verificaciôn.',
     },
     additionalDetailsStep: {
         headerTitle: 'Detalles adicionales',