fix: removed duplicate popup when cancel connection

Author: hugo-heer

src/providers/WalletProvider.tsx

@@ -48,20 +48,18 @@ export const WalletProvider = ({ children }: { children: React.ReactNode }) => {
       try {
         popupLock.current = true;
         wallet.setWallet(walletId);
-
         const [a, n] = await Promise.all([
           wallet.getAddress(),
           wallet.getNetwork(),
         ]);
-
+        if (!a.address) storage.setItem("walletId", "");
         if (a.address !== address) setAddress(a.address);
         if (n.network !== network) setNetwork(n.network);
         if (n.networkPassphrase !== networkPassphrase)
           setNetworkPassphrase(n.networkPassphrase);
       } catch (e) {
         // If `getNetwork` or `getAddress` throw errors... sign the user out???
         nullify();
-
         // then log the error (instead of throwing) so we have visibility
         // into the error while working on Scaffold Stellar but we do not
         // crash the app process

src/util/wallet.ts

@@ -22,10 +22,11 @@ export const connectWallet = async () => {
 
       // Now open selected wallet's login flow by calling `getAddress` --
       // Yes, it's strange that a getter has a side effect of opening a modal
-      void kit.getAddress().then(() => {
+      void kit.getAddress().then((address) => {
         // Once `getAddress` returns successfully, we know they actually
         // connected the selected wallet, and we set our localStorage
-        storage.setItem("walletId", selectedId);
+        if (address.address) storage.setItem("walletId", selectedId);
+        else storage.setItem("walletId", "");
       });
     },
   });