Merge pull request #4867 from BitGo/COIN-1404-fix-polygon

chore(sdk-coin-polygon): use nonpacked encoding for v4 wallet
BitGo · Aug 30, 2024 · 8ab5471 · 8ab5471
2 parents 696a3e7 + f6e9536
commit 8ab5471
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/modules/abstract-eth/src/lib/transactionBuilder.ts b/modules/abstract-eth/src/lib/transactionBuilder.ts
@@ -64,7 +64,7 @@ export abstract class TransactionBuilder extends BaseTransactionBuilder {
 
   // Wallet initialization transaction parameters
   private _walletOwnerAddresses: string[];
-  private _walletVersion: number;
+  protected _walletVersion: number;
 
   // flush tokens parameters
   private _forwarderAddress: string;

diff --git a/modules/sdk-coin-polygon/src/lib/transactionBuilder.ts b/modules/sdk-coin-polygon/src/lib/transactionBuilder.ts
@@ -31,6 +31,6 @@ export class TransactionBuilder extends EthLikeTransactionBuilder {
   public coinUsesNonPackedEncodingForTxData(): boolean {
     // This is because the contracts which have been deployed for
     // polygon amoy testnet use non-packed encoding for tx data
-    return this._common.chainIdBN().toString() === '80002';
+    return this._common.chainIdBN().toString() === '80002' || this._walletVersion === 4;
   }
 }