fix(bridge-ui): fix claim eth message processed (#13656)

taikoxyz · Apr 26, 2023 · 57f06cb · 57f06cb
1 parent 89439bb
commit 57f06cb
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/packages/bridge-ui/src/bridge/ERC20Bridge.ts b/packages/bridge-ui/src/bridge/ERC20Bridge.ts
@@ -162,6 +162,7 @@ export class ERC20Bridge implements Bridge {
       messageStatus === MessageStatus.Done ||
       messageStatus === MessageStatus.Failed
     ) {
+      // TODO: should be throw a different error when status is Failed?
       throw Error('message already processed');
     }
 

diff --git a/packages/bridge-ui/src/bridge/ETHBridge.ts b/packages/bridge-ui/src/bridge/ETHBridge.ts
@@ -104,12 +104,18 @@ export class ETHBridge implements Bridge {
       throw Error('message already processed');
     }
 
+    if (messageStatus === MessageStatus.Failed) {
+      throw Error('user can not process this, message has failed');
+    }
+
     const signerAddress = await opts.signer.getAddress();
 
     if (opts.message.owner.toLowerCase() !== signerAddress.toLowerCase()) {
       throw Error('user can not process this, it is not their message');
     }
 
+    // TODO: up to here we share same logic as ERC20Bridge
+
     if (messageStatus === MessageStatus.New) {
       const proofOpts = {
         srcChain: opts.message.srcChainId,