Merge pull request #266 from pshenmic/feat/redesign

Fix state switch handler in validators list
pshenmic · Sep 22, 2024 · 82306f6 · 82306f6
2 parents f8bfe28 + fba3396
commit 82306f6
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/packages/frontend/src/app/validators/Validators.js b/packages/frontend/src/app/validators/Validators.js
@@ -77,11 +77,10 @@ function Validators ({ defaultPage = 1, defaultPageSize, defaultIsActive }) {
     router.push(`${pathname}?${urlParameters.toString()}`, { scroll: false })
   }, [currentPage, pageSize, activeState])
 
-  const isActiveSwitchHandler = (newActiveState) => {
+  useEffect(() => {
     setCurrentPage(0)
-    setActiveState(newActiveState.toLowerCase())
-    fetchData(1, pageSize, newActiveState)
-  }
+    fetchData(1, pageSize, activeState)
+  }, [activeState])
 
   return (
     <Container
@@ -102,7 +101,7 @@ function Validators ({ defaultPage = 1, defaultPageSize, defaultIsActive }) {
               { title: 'Queued' }
             ]}
             defaultValue={activeState}
-            onChange={activeOption => isActiveSwitchHandler(activeOption)}
+            onChange={activeOption => setActiveState(activeOption.toLowerCase())}
           />
         </Box>