fix: too many re-renders

lifinance · Jul 18, 2022 · c68c558 · c68c558
1 parent 810fccb
commit c68c558
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/packages/widget/src/hooks/useSwapRoutes.ts b/packages/widget/src/hooks/useSwapRoutes.ts
@@ -120,11 +120,12 @@ export const useSwapRoutes = () => {
       (route) => route.id === currentRoute?.id,
     );
     const recommendedRoute = data?.routes[0];
-    // we don't want to set the current route again if it's already selected from existing routes
-    if (!isCurrentRouteInSet && currentRoute !== recommendedRoute) {
+    // we don't want to set the current route again on mount if it's already selected from existing routes
+    if (!isCurrentRouteInSet) {
       setCurrentRoute(recommendedRoute);
     }
-  }, [currentRoute, data?.routes, setCurrentRoute]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [data?.routes, setCurrentRoute]);
 
   return {
     routes: data?.routes,