fix: coderabbit

Kodylow · Oct 25, 2024 · 0826b79 · 0826b79
1 parent 10d4352
commit 0826b79
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 17 deletions.
diff --git a/apps/router/src/gateway-ui/Gateway.tsx b/apps/router/src/gateway-ui/Gateway.tsx
@@ -91,7 +91,11 @@ export const Gateway = () => {
             </TabList>
             <Divider orientation='vertical' />
             <TabPanels flex={1} width='100%'>
-              <TabPanel>{state.balances && <WalletCard />}</TabPanel>
+              {state.balances && (
+                <TabPanel>
+                  <WalletCard />
+                </TabPanel>
+              )}
               <TabPanel>
                 <LightningCard />
               </TabPanel>

diff --git a/apps/router/src/gateway-ui/components/walletModal/index.tsx b/apps/router/src/gateway-ui/components/walletModal/index.tsx
@@ -49,13 +49,14 @@ export const CreateButton: React.FC<{
   onClick: () => void;
   label: string;
   isLoading?: boolean;
-}> = ({ onClick, label, isLoading }) => (
+  isDisabled?: boolean;
+}> = ({ onClick, label, isLoading, isDisabled }) => (
   <Button
     onClick={onClick}
     size='lg'
     width='100%'
     isLoading={isLoading}
-    isDisabled={isLoading}
+    isDisabled={isDisabled}
   >
     {label}
   </Button>

diff --git a/apps/router/src/hooks/gateway/useGateway.tsx b/apps/router/src/hooks/gateway/useGateway.tsx
@@ -115,7 +115,10 @@ export const useLoadGateway = () => {
       fetchInfoAndConfigs();
       fetchBalances();
 
-      const interval = setInterval(fetchInfoAndConfigs, 5000);
+      const interval = setInterval(() => {
+        fetchInfoAndConfigs();
+        fetchBalances();
+      }, 5000);
       return () => clearInterval(interval);
     }
   }, [state.needsAuth, api, dispatch]);

diff --git a/apps/router/src/hooks/guardian/useGuardianSetup.tsx b/apps/router/src/hooks/guardian/useGuardianSetup.tsx
@@ -144,19 +144,25 @@ export const useHandleBackgroundGuardianSetupActions = (
   const [isPollingConsensus, setIsPollingConsensus] = useState(false);
 
   const fetchConsensusState = useCallback(async () => {
-    const consensusState = await api.getConsensusConfigGenParams();
-    dispatch({
-      type: SETUP_ACTION_TYPE.SET_OUR_CURRENT_ID,
-      payload: consensusState.our_current_id,
-    });
-    dispatch({
-      type: SETUP_ACTION_TYPE.SET_PEERS,
-      payload: Object.values(consensusState.consensus.peers),
-    });
-    dispatch({
-      type: SETUP_ACTION_TYPE.SET_CONFIG_GEN_PARAMS,
-      payload: consensusState.consensus,
-    });
+    try {
+      const consensusState = await api.getConsensusConfigGenParams();
+      dispatch({
+        type: SETUP_ACTION_TYPE.SET_OUR_CURRENT_ID,
+        payload: consensusState.our_current_id,
+      });
+      dispatch({
+        type: SETUP_ACTION_TYPE.SET_PEERS,
+        payload: Object.values(consensusState.consensus.peers),
+      });
+      dispatch({
+        type: SETUP_ACTION_TYPE.SET_CONFIG_GEN_PARAMS,
+        payload: consensusState.consensus,
+      });
+    } catch (error) {
+      throw new Error(
+        `Failed to fetch consensus state: ${(error as Error).message}`
+      );
+    }
   }, [api, dispatch]);
 
   useEffect(() => {