(Deposit/Withdraw) Fix Duplicate change network requests (#3688)

osmosis-labs · Aug 6, 2024 · e59fe52 · e59fe52
1 parent ee12007
commit e59fe52
Showing 1 changed file with 2 additions and 18 deletions.
diff --git a/packages/web/components/bridge/amount-screen.tsx b/packages/web/components/bridge/amount-screen.tsx
@@ -571,8 +571,10 @@ export const AmountScreen = observer(
       ) {
         const firstChain = supportedChains[0];
         setChain(firstChain);
+        checkChainAndConnectWallet(firstChain);
       }
     }, [
+      checkChainAndConnectWallet,
       direction,
       fromChain,
       isLoadingSupportedAssets,
@@ -593,24 +595,6 @@ export const AmountScreen = observer(
           );
     }, [direction, canonicalAsset, assetsInOsmosis, toAsset?.address]);
 
-    /**
-     * Only check the chain and connect the wallet if the quote is enabled,
-     * the asset transfers are not disabled, and the chain is set.
-     */
-    useEffect(() => {
-      const chain = direction === "deposit" ? fromChain : toChain;
-      if (chain && quote.enabled && !areAssetTransfersDisabled) {
-        checkChainAndConnectWallet(chain);
-      }
-    }, [
-      checkChainAndConnectWallet,
-      direction,
-      fromChain,
-      areAssetTransfersDisabled,
-      quote.enabled,
-      toChain,
-    ]);
-
     const onChangeCryptoInput = useCallback(
       (amount: string) => {
         if (isNil(fromAsset?.decimals)) return;