Adjust some memos to include dependencies, hide router behind a windo…

…w check
fluidity-money · Jun 20, 2024 · ea9a1c5 · ea9a1c5
1 parent b6e4c8c
commit ea9a1c5
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/web/src/app/stake/pool/confirm-withdraw/page.tsx b/web/src/app/stake/pool/confirm-withdraw/page.tsx
@@ -42,7 +42,7 @@ export default function ConfirmWithdrawLiquidity() {
         args: [token0.address, id, delta],
       });
     },
-    [writeContractUpdatePosition, token0AmountRaw, token0],
+    [delta, writeContractUpdatePosition, token0AmountRaw, token0],
   );
 
   // price of the current pool

diff --git a/web/src/app/stake/pool/page.tsx b/web/src/app/stake/pool/page.tsx
@@ -83,7 +83,10 @@ export default function PoolPage() {
   const { data: userData } = useGraphqlUser();
   const allPoolsData = useFragment(ManagePoolFragment, globalData?.pools);
   const positionsData_ = useFragment(PositionsFragment, userData?.getWallet)
-  const positionsData = useMemo(() => positionsData_?.positions.positions.filter(p => p.pool.address === id), [positionsData_])
+  const positionsData = useMemo(() =>
+    positionsData_?.positions.positions.filter(p => p.pool.address === id),
+    [id, positionsData_]
+  );
 
 
   const {

diff --git a/web/src/app/stake/pool/withdraw-liquidity/page.tsx b/web/src/app/stake/pool/withdraw-liquidity/page.tsx
@@ -51,9 +51,10 @@ export default function WithdrawLiquidity() {
   const params = useSearchParams();
 
   const positionId = params.get("positionId")
-  if (!positionId) {
-    router.back();
-  }
+
+  useEffect(() => {
+    if (!positionId && typeof window !== undefined) router.back();
+  }, [positionId, router]);
 
   const { address } = useAccount()