Merge pull request #53614 from Krishna2323/krishna2323/issue/53414

fix: the 'connect to Xero' experience in two ways (modal style & translation copy update).
Expensify · Dec 10, 2024 · 9fe5511 · 9fe5511
2 parents df7ab48 + 3425965
commit 9fe5511
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 12 deletions.
diff --git a/src/components/RequireTwoFactorAuthenticationModal.tsx b/src/components/RequireTwoFactorAuthenticationModal.tsx
@@ -45,20 +45,12 @@ function RequireTwoFactorAuthenticationModal({onCancel = () => {}, description,
                     onClose={onCancel}
                     isVisible={isVisible}
                     type={shouldUseNarrowLayout ? CONST.MODAL.MODAL_TYPE.BOTTOM_DOCKED : CONST.MODAL.MODAL_TYPE.CONFIRM}
-                    innerContainerStyle={{...styles.pb5, ...styles.pt3, ...styles.boxShadowNone}}
+                    innerContainerStyle={{...styles.pb5, ...styles.pt0, ...styles.boxShadowNone}}
                     shouldEnableNewFocusManagement={shouldEnableNewFocusManagement}
                     animationOutTiming={500}
                 >
                     <View style={safeAreaPaddingBottomStyle}>
-                        <View
-                            style={[
-                                styles.mh3,
-                                styles.br3,
-                                styles.cardSectionIllustration,
-                                styles.alignItemsCenter,
-                                StyleUtils.getBackgroundColorStyle(LottieAnimations.Safe.backgroundColor),
-                            ]}
-                        >
+                        <View style={[styles.cardSectionIllustration, styles.alignItemsCenter, StyleUtils.getBackgroundColorStyle(LottieAnimations.Safe.backgroundColor)]}>
                             <Lottie
                                 source={LottieAnimations.Safe}
                                 style={styles.h100}

diff --git a/src/languages/en.ts b/src/languages/en.ts
@@ -1316,7 +1316,7 @@ const translations = {
         disable: 'Disable',
         enableTwoFactorAuth: 'Enable two-factor authentication',
         pleaseEnableTwoFactorAuth: 'Please enable two-factor authentication.',
-        twoFactorAuthIsRequiredDescription: 'Two-factor authentication is required for connecting to Xero. Please enable two-factor authentication to continue.',
+        twoFactorAuthIsRequiredDescription: 'For security purposes, Xero requires two-factor authentication to connect the integration.',
         twoFactorAuthIsRequiredForAdminsDescription: 'Two-factor authentication is required for Xero workspace admins. Please enable two-factor authentication to continue.',
     },
     recoveryCodeForm: {

diff --git a/src/languages/es.ts b/src/languages/es.ts
@@ -1316,7 +1316,7 @@ const translations = {
         disable: 'Deshabilitar',
         enableTwoFactorAuth: 'Activar la autenticación de dos factores',
         pleaseEnableTwoFactorAuth: 'Activa la autenticación de dos factores.',
-        twoFactorAuthIsRequiredDescription: 'La autenticación de dos factores es necesaria para conectarse a Xero. Activa la autenticación de dos factores para continuar.',
+        twoFactorAuthIsRequiredDescription: 'Por razones de seguridad, Xero requiere la autenticación de dos factores para conectar la integración.',
         twoFactorAuthIsRequiredForAdminsDescription:
             'La autenticación de dos factores es necesaria para los administradores del área de trabajo de Xero. Activa la autenticación de dos factores para continuar.',
     },