Fix sign-in request branch case for mobile devices when signing in wi…

…th mobile wallets (#6704) * Fix auto auth case for mobile devices * Fix page refresh on landing page
hicommonwealth · Feb 12, 2024 · c05f6e9 · c05f6e9
1 parent 14d979f
commit c05f6e9
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 3 deletions.
diff --git a/packages/commonwealth/client/scripts/hooks/useWallets.tsx b/packages/commonwealth/client/scripts/hooks/useWallets.tsx
@@ -641,7 +641,16 @@ const useWallets = (walletProps: IuseWalletProps) => {
         setSignerAccount(signingAccount);
         setIsNewlyCreated(newlyCreated);
         setIsLinkingOnMobile(isLinkingWallet);
-        setActiveStep('redirectToSign');
+        if (featureFlags.newSignInModal) {
+          onAccountVerified(
+            signingAccount,
+            newlyCreated,
+            isLinkingWallet,
+            wallet,
+          );
+        } else {
+          setActiveStep('redirectToSign');
+        }
       } else {
         onAccountVerified(
           signingAccount,
@@ -724,7 +733,11 @@ const useWallets = (walletProps: IuseWalletProps) => {
       if (setSignerAccount) setSignerAccount(account);
       if (setIsNewlyCreated) setIsNewlyCreated(false);
       if (setIsLinkingOnMobile) setIsLinkingOnMobile(false);
-      setActiveStep('redirectToSign');
+      if (featureFlags.newSignInModal) {
+        onAccountVerified(account, false, false);
+      } else {
+        setActiveStep('redirectToSign');
+      }
       return;
     } else {
       onAccountVerified(account, false, false);

diff --git a/packages/commonwealth/client/scripts/views/pages/landing/header.tsx b/packages/commonwealth/client/scripts/views/pages/landing/header.tsx
@@ -61,8 +61,9 @@ export const Header = ({ onLogin }: HeaderProps) => {
         />
       ) : (
         <AuthModal
-          onClose={() => setIsAuthModalOpen(false)}
+          onSuccess={onLogin}
           isOpen={isAuthModalOpen}
+          onClose={() => setIsAuthModalOpen(false)}
         />
       )}
     </>