Merge pull request #53297 from nyomanjyotisa/issue-52086-2

Fix: SMS delivery failure sign in flow in homepage
Expensify · Nov 30, 2024 · 32ce9d4 · 32ce9d4
2 parents 6d56a85 + 15182a8
commit 32ce9d4
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/pages/signin/SMSDeliveryFailurePage.tsx b/src/pages/signin/SMSDeliveryFailurePage.tsx
@@ -26,7 +26,7 @@ function SMSDeliveryFailurePage() {
         return Str.isSMSLogin(credentials.login) ? Str.removeSMSDomain(credentials.login) : credentials.login;
     }, [credentials?.login]);
 
-    const SMSDeliveryFailureMessage = account?.SMSDeliveryFailureStatus?.message;
+    const SMSDeliveryFailureMessage = account?.smsDeliveryFailureStatus?.message;
 
     useEffect(() => {
         if (!isKeyboardShown) {

diff --git a/src/pages/signin/SignInPage.tsx b/src/pages/signin/SignInPage.tsx
@@ -94,7 +94,7 @@ function getRenderOptions({
     const isSAMLEnabled = !!account?.isSAMLEnabled;
     const isSAMLRequired = !!account?.isSAMLRequired;
     const hasEmailDeliveryFailure = !!account?.hasEmailDeliveryFailure;
-    const hasSMSDeliveryFailure = !!account?.SMSDeliveryFailureStatus?.hasSMSDeliveryFailure;
+    const hasSMSDeliveryFailure = !!account?.smsDeliveryFailureStatus?.hasSMSDeliveryFailure;
 
     // True, if the user has SAML required, and we haven't yet initiated SAML for their account
     const shouldInitiateSAMLLogin = hasAccount && hasLogin && isSAMLRequired && !hasInitiatedSAMLLogin && !!account.isLoading;

diff --git a/src/types/onyx/Account.ts b/src/types/onyx/Account.ts
@@ -156,7 +156,7 @@ type Account = {
     delegatedAccess?: DelegatedAccess;
 
     /** Indicates SMS delivery failure status and associated information */
-    SMSDeliveryFailureStatus?: SMSDeliveryFailureStatus;
+    smsDeliveryFailureStatus?: SMSDeliveryFailureStatus;
 };
 
 export default Account;