fix: reduce initialization requests in strict mode

lifinance · Nov 9, 2022 · 1892674 · 1892674
1 parent 1bedc9e
commit 1892674
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 21 deletions.
diff --git a/packages/widget/src/hooks/useInitializer.ts b/packages/widget/src/hooks/useInitializer.ts
@@ -4,9 +4,14 @@ import { useEffect } from 'react';
 import { name, version } from '../config/version';
 import { useTools } from './useTools';
 
+let checkedPackageUpdates = false;
+
 export const useInitializer = () => {
   useTools();
   useEffect(() => {
-    checkPackageUpdates(name, version);
+    if (!checkedPackageUpdates) {
+      checkedPackageUpdates = true;
+      checkPackageUpdates(name, version);
+    }
   }, []);
 };
diff --git a/packages/widget/src/hooks/useTools.ts b/packages/widget/src/hooks/useTools.ts
@@ -10,27 +10,23 @@ type FormattedTool<T, K extends keyof T> = Record<string, Pick<T, K>>;
 export const useTools = () => {
   const lifi = useLiFi();
   const { bridges, exchanges } = useWidgetConfig();
-  const { data } = useQuery(
-    ['tools'],
-    ({ signal }) => lifi.getTools(undefined, { signal }),
-    {
-      onSuccess(data) {
-        const { initializeTools } = useSettingsStore.getState();
-        initializeTools(
-          'Bridges',
-          data.bridges
-            .filter((bridge) => isItemAllowed(bridge.key, bridges))
-            .map((bridge) => bridge.key),
-        );
-        initializeTools(
-          'Exchanges',
-          data.exchanges
-            .filter((exchange) => isItemAllowed(exchange.key, exchanges))
-            .map((exchange) => exchange.key),
-        );
-      },
+  const { data } = useQuery(['tools'], () => lifi.getTools(), {
+    onSuccess(data) {
+      const { initializeTools } = useSettingsStore.getState();
+      initializeTools(
+        'Bridges',
+        data.bridges
+          .filter((bridge) => isItemAllowed(bridge.key, bridges))
+          .map((bridge) => bridge.key),
+      );
+      initializeTools(
+        'Exchanges',
+        data.exchanges
+          .filter((exchange) => isItemAllowed(exchange.key, exchanges))
+          .map((exchange) => exchange.key),
+      );
     },
-  );
+  });
 
   const formattedTools = useMemo(
     () => ({