Fix the verify() step of the tx flows updating troves (#724)

liquity · Jan 20, 2025 · 1189ab8 · 1189ab8
1 parent f591ee5
commit 1189ab8
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 79 deletions.
diff --git a/frontend/app/src/tx-flows/shared.ts b/frontend/app/src/tx-flows/shared.ts
@@ -1,4 +1,4 @@
-import type { CollIndex } from "@/src/types";
+import type { CollIndex, TroveId } from "@/src/types";
 import type { Config as WagmiConfig } from "wagmi";
 
 import { getPrefixedTroveId } from "@/src/liquity-utils";
@@ -54,17 +54,23 @@ export async function verifyTransaction(
 export async function verifyTroveUpdate(
   wagmiConfig: WagmiConfig,
   hash: string,
-  collIndex: CollIndex,
-  lastUpdate: number,
+  loan: {
+    collIndex: CollIndex;
+    troveId: TroveId;
+    updatedAt: number;
+  },
 ) {
-  const receipt = await waitForTransactionReceipt(wagmiConfig, {
+  await waitForTransactionReceipt(wagmiConfig, {
     hash: hash as `0x${string}`,
   });
-  const prefixedTroveId = getPrefixedTroveId(collIndex, receipt.transactionHash);
+  const prefixedTroveId = getPrefixedTroveId(loan.collIndex, loan.troveId);
   while (true) {
     // wait for the trove to be updated in the subgraph
-    const { trove } = await graphQuery(TroveByIdQuery, { id: prefixedTroveId });
-    if (trove && Number(trove.updatedAt) * 1000 !== lastUpdate) {
+    const { trove } = await graphQuery(
+      TroveByIdQuery,
+      { id: prefixedTroveId },
+    );
+    if (trove && Number(trove.updatedAt) * 1000 !== loan.updatedAt) {
       break;
     }
     await sleep(1000);

diff --git a/frontend/app/src/tx-flows/updateBorrowPosition.tsx b/frontend/app/src/tx-flows/updateBorrowPosition.tsx
@@ -262,12 +262,7 @@ export const updateBorrowPosition: FlowDeclaration<UpdateBorrowPositionRequest>
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
 
@@ -299,12 +294,7 @@ export const updateBorrowPosition: FlowDeclaration<UpdateBorrowPositionRequest>
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
 
@@ -337,12 +327,7 @@ export const updateBorrowPosition: FlowDeclaration<UpdateBorrowPositionRequest>
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
 
@@ -374,12 +359,7 @@ export const updateBorrowPosition: FlowDeclaration<UpdateBorrowPositionRequest>
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
 
@@ -411,12 +391,7 @@ export const updateBorrowPosition: FlowDeclaration<UpdateBorrowPositionRequest>
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
   },

diff --git a/frontend/app/src/tx-flows/updateLeveragePosition.tsx b/frontend/app/src/tx-flows/updateLeveragePosition.tsx
@@ -260,12 +260,7 @@ export const updateLeveragePosition: FlowDeclaration<UpdateLeveragePositionReque
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
 
@@ -303,12 +298,7 @@ export const updateLeveragePosition: FlowDeclaration<UpdateLeveragePositionReque
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
 
@@ -361,12 +351,7 @@ export const updateLeveragePosition: FlowDeclaration<UpdateLeveragePositionReque
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
 
@@ -416,12 +401,7 @@ export const updateLeveragePosition: FlowDeclaration<UpdateLeveragePositionReque
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
   },

diff --git a/frontend/app/src/tx-flows/updateLoanInterestRate.tsx b/frontend/app/src/tx-flows/updateLoanInterestRate.tsx
@@ -201,12 +201,7 @@ export const updateLoanInterestRate: FlowDeclaration<UpdateLoanInterestRateReque
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
 
@@ -241,12 +236,7 @@ export const updateLoanInterestRate: FlowDeclaration<UpdateLoanInterestRateReque
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
 
@@ -277,12 +267,7 @@ export const updateLoanInterestRate: FlowDeclaration<UpdateLoanInterestRateReque
       },
 
       async verify({ request, wagmiConfig }, hash) {
-        await verifyTroveUpdate(
-          wagmiConfig,
-          hash,
-          request.loan.collIndex,
-          request.loan.updatedAt,
-        );
+        await verifyTroveUpdate(wagmiConfig, hash, request.loan);
       },
     },
   },