Update to pass lifeCycleStatus instead of maxSlippage to processSwapT…

…ransaction
coinbase · Sep 5, 2024 · cab4738 · cab4738
1 parent fce027d
commit cab4738
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/src/swap/components/SwapProvider.tsx b/src/swap/components/SwapProvider.tsx
@@ -280,7 +280,7 @@ export function SwapProvider({
       }
       await processSwapTransaction({
         config,
-        maxSlippage,
+        lifeCycleStatus,
         sendTransactionAsync,
         setLifeCycleStatus,
         swapTransaction: response,

diff --git a/src/swap/types.ts b/src/swap/types.ts
@@ -117,11 +117,11 @@ export type LifeCycleStatus =
 
 export type ProcessSwapTransactionParams = {
   config: Config;
+  lifeCycleStatus: LifeCycleStatus;
   setLifeCycleStatus: (state: LifeCycleStatus) => void;
   sendTransactionAsync: SendTransactionMutateAsync<Config, unknown>;
   swapTransaction: BuildSwapTransaction;
   useAggregator: boolean;
-  maxSlippage: number;
 };
 
 /**

diff --git a/src/swap/utils/processSwapTransaction.ts b/src/swap/utils/processSwapTransaction.ts
@@ -9,14 +9,19 @@ import type { ProcessSwapTransactionParams } from '../types';
 
 export async function processSwapTransaction({
   config,
-  maxSlippage,
+  lifeCycleStatus,
   sendTransactionAsync,
   setLifeCycleStatus,
   swapTransaction,
   useAggregator,
 }: ProcessSwapTransactionParams) {
   const { transaction, approveTransaction, quote } = swapTransaction;
 
+  const maxSlippage =
+    lifeCycleStatus.statusName !== 'error'
+      ? lifeCycleStatus.statusData.maxSlippage
+      : 3;
+
   // for swaps from ERC-20 tokens,
   // if there is an approveTransaction present,
   // request approval for the amount