Fix: Catch for no provider

JoinColony · Feb 23, 2023 · 56ff4c6 · 56ff4c6
1 parent 1524e0b
commit 56ff4c6
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 23 deletions.
diff --git a/src/modules/dashboard/sagas/utils/safeHelpers.ts b/src/modules/dashboard/sagas/utils/safeHelpers.ts
@@ -101,7 +101,9 @@ export const getHomeBridgeByChain = (safeChainId: string) => {
     );
   }
   // @ts-ignore abi type is wrong.
-  return new ethers.Contract(homeBridgeAddress, HomeAMB, homeSigner);
+  return homeSigner
+    ? new ethers.Contract(homeBridgeAddress, HomeAMB, homeSigner)
+    : null;
 };
 
 const getErc721 = (

diff --git a/src/utils/safes/getTransactionStatuses.ts b/src/utils/safes/getTransactionStatuses.ts
@@ -110,28 +110,31 @@ export const getTransactionStatuses = async (
   const homeAMBContract = getHomeBridgeByChain(safeChainId);
   const foreignAMBContract = getForeignBridgeByChain(safeChainId);
 
-  const messageIds = getMessageIds(
-    transactionReceipt,
-    homeAMBContract,
-    homeAMBContract.address,
-  );
+  if (homeAMBContract) {
+    const messageIds = getMessageIds(
+      transactionReceipt,
+      homeAMBContract,
+      homeAMBContract?.address,
+    );
+    const transactionStatuses = await Promise.all(
+      messageIds.map(async (messageId) => {
+        const wasTheMessageDelivered = await checkIfTheMessageWasDelivered(
+          foreignAMBContract,
+          networkApiURI,
+          messageId,
+          safeChainId,
+        );
+
+        // @NOTE: Safe transactions will always be executed automatically on the local env
+        if (wasTheMessageDelivered || onLocalDevEnvironment) {
+          return TRANSACTION_STATUS.SUCCESS;
+        }
+        return TRANSACTION_STATUS.PENDING;
+      }),
+    );
 
-  const transactionStatuses = await Promise.all(
-    messageIds.map(async (messageId) => {
-      const wasTheMessageDelivered = await checkIfTheMessageWasDelivered(
-        foreignAMBContract,
-        networkApiURI,
-        messageId,
-        safeChainId,
-      );
-
-      // @NOTE: Safe transactions will always be executed automatically on the local env
-      if (wasTheMessageDelivered || onLocalDevEnvironment) {
-        return TRANSACTION_STATUS.SUCCESS;
-      }
-      return TRANSACTION_STATUS.PENDING;
-    }),
-  );
+    return transactionStatuses;
+  }
 
-  return transactionStatuses;
+  return [];
 };