fix: use correct destination chain and address for ibc withdraw (#2016)

Author: mateuszjasiuk

apps/namadillo/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@namada/namadillo",
-  "version": "1.20.0",
+  "version": "1.20.0-hotfix2",
   "description": "Namadillo",
   "repository": "https://github.com/anoma/namada-interface/",
   "author": "Heliax Dev <[email protected]>",

apps/namadillo/src/App/Ibc/IbcWithdraw.tsx

@@ -130,21 +130,24 @@ export const IbcWithdraw = (): JSX.Element => {
     }
   };
 
-  const updateDestinationChainAndAddress = (chain: Chain | undefined): void => {
+  const updateDestinationChainAndAddress = async (
+    chain: Chain | undefined
+  ): Promise<void> => {
     setDestinationChain(chain);
     if (customAddress) {
       setCustomAddress("");
     }
     if (chain) {
-      loadWalletAddress(chain?.chain_id);
+      await connectToChainId(chain.chain_id);
+      await loadWalletAddress(chain.chain_id);
     }
   };
 
   const {
     data: ibcChannels,
     isError: unknownIbcChannels,
     isLoading: isLoadingIbcChannels,
-  } = useAtomValue(ibcChannelsFamily(registry?.chain.chain_name));
+  } = useAtomValue(ibcChannelsFamily(destinationChain?.chain_name));
 
   useEffect(() => {
     setSourceChannel(ibcChannels?.namadaChannel || "");
@@ -154,23 +157,25 @@ export const IbcWithdraw = (): JSX.Element => {
   // to other chains different than the original one. Ex: OSMO should only be withdrew to Osmosis,
   // ATOM to Cosmoshub, etc.
   useEffect(() => {
-    if (!selectedAsset || !chainTokens.data) {
-      updateDestinationChainAndAddress(undefined);
-      return;
-    }
+    (async () => {
+      if (!selectedAsset || !chainTokens.data) {
+        await updateDestinationChainAndAddress(undefined);
+        return;
+      }
 
-    const token = chainTokens.data.find(
-      (token) => token.address === selectedAsset.originalAddress
-    );
+      const token = chainTokens.data.find(
+        (token) => token.address === selectedAsset.originalAddress
+      );
 
-    if (token && "trace" in token) {
-      const denom = getDenomFromIbcTrace(token.trace);
-      const chain = searchChainByDenom(denom);
-      updateDestinationChainAndAddress(chain);
-      return;
-    }
+      if (token && "trace" in token) {
+        const denom = getDenomFromIbcTrace(token.trace);
+        const chain = searchChainByDenom(denom);
+        await updateDestinationChainAndAddress(chain);
+        return;
+      }
 
-    updateDestinationChainAndAddress(undefined);
+      await updateDestinationChainAndAddress(undefined);
+    })();
   }, [selectedAsset, chainTokens.data]);
 
   const {

apps/namadillo/src/hooks/useWalletManager.tsx

@@ -29,12 +29,14 @@ export const useWalletManager = (wallet: WalletConnector): UseWalletOutput => {
   const isConnected = connectedWallets[walletKey];
 
   useEffect(() => {
-    if (isConnected && chainId) {
-      connectToChainId(chainId);
-    } else {
-      setWalletAddress(undefined);
-      setRegistry(undefined);
-    }
+    (async () => {
+      if (isConnected && chainId) {
+        await connectToChainId(chainId);
+      } else {
+        setWalletAddress(undefined);
+        setRegistry(undefined);
+      }
+    })();
   }, [isConnected, walletKey, chainId]);
 
   const connectToChainId = async (chainId: string): Promise<void> => {