Fixed Pending Confirmations Modal (#106)

## Pull Request Title: Improved transaction verification in verifyTx function ## Fixes: #104 ## Issue: The original verifyTx function only checked the transaction status once after a fixed delay, leading to unreliable verification. ## Fix: Refactored the verifyTx function to continuously check the transaction status until confirmation or rejection. Implemented recursive checking using setTimeout for regular status updates. --------- Co-authored-by: Bruno <[email protected]>
DjedAlliance · Apr 23, 2024 · 38a657f · 38a657f
1 parent 9f778d9
commit 38a657f
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 58 deletions.
diff --git a/craco.config.js b/craco.config.js
@@ -32,4 +32,4 @@ module.exports = {
       return webpackConfig;
     }
   }
-};
+};
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/utils/ethereum.js b/src/utils/ethereum.js
@@ -230,9 +230,14 @@ export const promiseTx = (isWalletConnected, tx, signer) => {
 
 export const verifyTx = (web3, hash) => {
   return new Promise((res) => {
-    setTimeout(() => {
-      web3.eth.getTransactionReceipt(hash).then((receipt) => res(receipt.status));
-    }, CONFIRMATION_WAIT_PERIOD);
+    const checkStatus = () => {
+      web3.eth.getTransactionReceipt(hash).then((receipt) => {
+        receipt != null ?
+          res(receipt.status) :
+          setTimeout(checkStatus, CONFIRMATION_WAIT_PERIOD);
+      });
+    };
+    checkStatus();
   });
 };
-Original file line number
+Diff line change
@@ Expand Up / @@ -32,4 +32,4 @@ module.exports = { @@
           return webpackConfig;
         }
       }
-    };
+    };