⛽ Settle safeTxGas (#1139)

TrueFiEng · Sep 4, 2023 · a02c280 · a02c280 · github-actions · Sep 4, 2023
1 parent 95ed92f
commit a02c280
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/packages/core/src/helpers/gnosisSafeUtils.ts b/packages/core/src/helpers/gnosisSafeUtils.ts
@@ -77,6 +77,11 @@ export const calculateSafeTransactionHash = (
   safeTx: SafeTransaction,
   chainId: BigNumberish
 ): string => {
+  // TODO: Remove after WalletConnectV2 fix
+  // WalletConnectV2 incorrectly passes safeTxGas value, equal 0 (default value).
+  // Thus we have a mismatch between gnosis safeTxHash and one calculated by useDApp (safeTxGas is based on estimation)
+  safeTx = { ...safeTx, safeTxGas: 0 }
+
   return utils._TypedDataEncoder.hash({ verifyingContract: safe.address, chainId }, EIP712_SAFE_TX_TYPE, safeTx)
 }