fix(livesdklogic): adjust gas setting to be on par with lld

LedgerHQ · Apr 26, 2024 · c67871c · c67871c
1 parent 882c77d
commit c67871c
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 14 deletions.
diff --git a/.changeset/old-teachers-march.md b/.changeset/old-teachers-march.md
@@ -0,0 +1,6 @@
+---
+"live-mobile": minor
+---
+
+Standardize PrepareSignTx behavior\
+
diff --git a/apps/ledger-live-mobile/src/components/Web3AppWebview/liveSDKLogic.ts b/apps/ledger-live-mobile/src/components/Web3AppWebview/liveSDKLogic.ts
@@ -6,22 +6,15 @@ import BigNumber from "bignumber.js";
 
 export default function prepareSignTransaction(
   account: AccountLike,
-  parentAccount: Account | undefined,
-  liveTx: Partial<Transaction & { gasLimit: BigNumber }>,
+  parentAccount: Account | null | undefined,
+  liveTx: Partial<
+    Transaction & {
+      gasLimit: BigNumber;
+    }
+  >,
 ): TransactionCommon {
   const bridge = getAccountBridge(account, parentAccount);
-  const t = bridge.createTransaction(account);
-  const { recipient, ...txData } = liveTx;
-  const t2 = bridge.updateTransaction(t, {
-    recipient,
+  return bridge.updateTransaction(liveTx, {
     subAccountId: isTokenAccount(account) ? account.id : undefined,
   });
-
-  return bridge.updateTransaction(t2, {
-    customGasLimit: txData.gasLimit,
-    type: 1,
-    maxFeePerGas: undefined,
-    maxPriorityFeePerGas: undefined,
-    ...txData,
-  });
 }