fix: remove customData when sending the tx via ethers

0xflair · Dec 8, 2022 · 3031dc5 · 3031dc5
1 parent cbf90c2
commit 3031dc5
Showing 1 changed file with 16 additions and 4 deletions.
diff --git a/packages/react/src/core/balance-ramp/services/balance-ramp.client.ts b/packages/react/src/core/balance-ramp/services/balance-ramp.client.ts
@@ -74,14 +74,14 @@ export class BalanceRampClient {
     });
 
     if (!requiredBalance?.outputTokenAddress) {
-      return originalSigner.sendTransaction(txWithGasData);
+      return this.sendTransactionOriginal(originalSigner, txWithGasData);
     }
 
     if (
       requiredBalance.outputTokenAddress !== constants.AddressZero &&
       !ethers.utils.isAddress(requiredBalance.outputTokenAddress)
     ) {
-      return originalSigner.sendTransaction(txWithGasData);
+      return this.sendTransactionOriginal(originalSigner, txWithGasData);
     }
 
     const currentBalance = await this.getCurrentBalance(
@@ -153,10 +153,22 @@ export class BalanceRampClient {
         );
       }
 
-      return originalSigner.sendTransaction(txWithGasData);
+      return this.sendTransactionOriginal(originalSigner, txWithGasData);
     }
 
-    return originalSigner.sendTransaction(txWithGasData);
+    return this.sendTransactionOriginal(originalSigner, txWithGasData);
+  }
+
+  async sendTransactionOriginal(
+    originalSigner: ethers.Signer,
+    txWithGasData: ethers.utils.Deferrable<TransactionRequest>,
+  ): Promise<TransactionResponse> {
+    const cleanTx = {
+      ...txWithGasData,
+      customData: undefined,
+    };
+
+    return originalSigner.sendTransaction(cleanTx);
   }
 
   async handleEstimateGasLimit(