Interest rates brackets: avoid extra requests

liquity · Jan 22, 2025 · 6ab7ad3 · 6ab7ad3
1 parent 9d5e8d0
commit 6ab7ad3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/frontend/app/src/liquity-utils.ts b/frontend/app/src/liquity-utils.ts
@@ -6,7 +6,7 @@ import type { Config as WagmiConfig } from "wagmi";
 
 import { DATA_REFRESH_INTERVAL, INTEREST_RATE_INCREMENT, INTEREST_RATE_MAX, INTEREST_RATE_MIN } from "@/src/constants";
 import { getCollateralContract, getContracts, getProtocolContract } from "@/src/contracts";
-import { dnum18 } from "@/src/dnum-utils";
+import { dnum18, jsonStringifyWithDnum } from "@/src/dnum-utils";
 import { CHAIN_BLOCK_EXPLORER } from "@/src/env";
 import {
   calculateStabilityPoolApr,
@@ -333,8 +333,7 @@ export function useInterestRateChartData(collIndex: null | CollIndex) {
     queryKey: [
       "useInterestRateChartData",
       collIndex,
-      brackets.status,
-      brackets.dataUpdatedAt,
+      jsonStringifyWithDnum(brackets.data),
     ],
     queryFn: () => {
       if (!brackets.isSuccess) {

diff --git a/frontend/app/src/subgraph-hooks.ts b/frontend/app/src/subgraph-hooks.ts
@@ -14,6 +14,7 @@ import { sleep } from "@/src/utils";
 import { isAddress, shortenAddress } from "@liquity2/uikit";
 import { useQuery } from "@tanstack/react-query";
 import * as dn from "dnum";
+import { useCallback } from "react";
 import {
   AllInterestRateBracketsQuery,
   BorrowerInfoQuery,
@@ -402,7 +403,7 @@ export function useInterestRateBrackets(
   return useQuery({
     queryKey: ["AllInterestRateBrackets"],
     queryFn,
-    select: (brackets) => {
+    select: useCallback((brackets: Awaited<ReturnType<typeof queryFn>>) => {
       // only filter by collIndex in the select()
       // so that we can query all the brackets at once
       return brackets
@@ -412,7 +413,7 @@ export function useInterestRateBrackets(
           rate: dnum18(bracket.rate),
           totalDebt: dnum18(bracket.totalDebt),
         }));
-    },
+    }, []),
     ...prepareOptions(options),
   });
 }