fix: modal states

masa-finance · Oct 26, 2023 · 309b582 · 309b582
1 parent fc298f3
commit 309b582
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/src/ui/components/modals/authenticate/authenticate.tsx b/src/ui/components/modals/authenticate/authenticate.tsx
@@ -26,7 +26,6 @@ export const Authenticate = ({
     disconnect,
     openConnectModal,
     isLoadingSigner,
-    hasAddress,
   } = useWallet();
 
   const [copied, setCopied] = useState(false);
@@ -51,6 +50,7 @@ export const Authenticate = ({
     needsWalletConnection,
     showAuthenticateView,
     showSwitchWalletButton,
+    showConnectedView,
   } = useAuthenticateModal({
     onAuthenticateError,
     onAuthenticateSuccess,
@@ -62,7 +62,7 @@ export const Authenticate = ({
     }
 
     let timeout: NodeJS.Timeout;
-    if (modal.visible && !isAuthenticating && hasAddress) {
+    if (modal.visible && !isAuthenticating && showConnectedView) {
       timeout = setTimeout(() => {
         modal.remove();
         // onClose?.();
@@ -75,7 +75,7 @@ export const Authenticate = ({
   }, [
     needsWalletConnection,
     modal,
-    hasAddress,
+    showConnectedView,
     isAuthenticating,
     openConnectModal,
   ]);

diff --git a/src/ui/components/modals/authenticate/use-authenticate-modal.ts b/src/ui/components/modals/authenticate/use-authenticate-modal.ts
@@ -29,6 +29,11 @@ export const useAuthenticateModal = ({
     [isConnected, hasSession, signer, hasAddress, isLoadingSession]
   );
 
+  const showConnectedView = useMemo(
+    () => hasSession && hasAddress,
+    [hasSession, hasAddress]
+  );
+
   const successMessage = useMemo(() => {
     switch (company) {
       case 'Masa': {
@@ -80,7 +85,7 @@ export const useAuthenticateModal = ({
     needsWalletConnection,
     showAuthenticateView,
     successMessage,
-
+    showConnectedView,
     isAuthenticating,
     onAuthenticateStart,
     showSwitchWalletButton,