refactor: unwrap TX on L2 use runWithTransactionLogger

lidofinance · Oct 1, 2024 · 9b0b4ee · 9b0b4ee
1 parent bd0d7d9
commit 9b0b4ee
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 8 deletions.
diff --git a/features/wsteth/unwrap/hooks/use-unwrap-form-processing.ts b/features/wsteth/unwrap/hooks/use-unwrap-form-processing.ts
@@ -83,14 +83,23 @@ export const useUnwrapFormProcessor = ({
 
         let txHash: string;
         if (isSDKSupportedL2Chain(chainId as CHAINS)) {
+          // TODO: remove without runWithTransactionLogger
+          // txHash = (
+          //   await sdk.l2.wrapWstethToSteth({
+          //     value: amount.toBigInt(),
+          //   })
+          // ).hash;
           txHash = (
-            await sdk.l2.wrapWstethToSteth({
-              value: amount.toBigInt(),
-            })
+            await runWithTransactionLogger('Unwrap signing on L2', () =>
+              // The operation 'wstETH to stETH' on L2 is 'wrap'
+              sdk.l2.wrapWstethToSteth({
+                value: amount.toBigInt(),
+              }),
+            )
           ).hash;
         } else {
           txHash = await runWithTransactionLogger(
-            'Unwrap signing',
+            'Unwrap signing on L1',
             async () => {
               const tx =
                 await wstethContractWeb3.populateTransaction.unwrap(amount);

diff --git a/features/wsteth/unwrap/hooks/use-unwrap-tx-on-l2-approve.ts b/features/wsteth/unwrap/hooks/use-unwrap-tx-on-l2-approve.ts
@@ -1,6 +1,7 @@
 import { useMemo, useState, useEffect, useCallback } from 'react';
 import type { BigNumber } from 'ethers';
 
+import { runWithTransactionLogger } from 'utils';
 import { useLidoSDK } from 'providers/lido-sdk';
 
 type UseUnwrapTxApproveArgs = {
@@ -30,10 +31,18 @@ export const useUnwrapTxOnL2Approve = ({ amount }: UseUnwrapTxApproveArgs) => {
   const processApproveTx = useCallback(
     async ({ onTxSent }) => {
       try {
+        // TODO: remove without runWithTransactionLogger
+        // const approveTxHash = (
+        //   await sdk.l2.approveWstethForWrap({
+        //     value: amount.toBigInt(),
+        //   })
+        // ).hash;
         const approveTxHash = (
-          await sdk.l2.approveWstethForWrap({
-            value: amount.toBigInt(),
-          })
+          await runWithTransactionLogger('Approve signing on L2', () =>
+            sdk.l2.approveWstethForWrap({
+              value: amount.toBigInt(),
+            }),
+          )
         ).hash;
 
         await onTxSent?.(approveTxHash);

diff --git a/shared/hooks/useApproveOnL1.ts b/shared/hooks/useApproveOnL1.ts
@@ -75,7 +75,7 @@ export const useApproveOnL1 = (
       };
 
       const approveTxHash = await runWithTransactionLogger(
-        'Approve signing',
+        'Approve signing on L1',
         processApproveTx,
       );
       await onTxSent?.(approveTxHash);