Skip to content

Commit

Permalink
fix: preHook fundToken alloance and approval
Browse files Browse the repository at this point in the history
  • Loading branch information
jmdev3 committed Jun 24, 2024
1 parent 584bc65 commit 28c5eab
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/handlers/evm/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -309,7 +309,7 @@ export class EvmHandler extends Utils {
params: RouteRequest,
signer?: EvmWallet,
): RouteParamsPopulated {
const { fromChain, toChain, fromToken, toToken } = params;
const { fromChain, toChain, fromToken, toToken, preHook } = params;

const _fromChain = tokensChains.getChainData(fromChain);
const _toChain = tokensChains.getChainData(toChain);
Expand All @@ -322,8 +322,11 @@ export class EvmHandler extends Utils {
let fromTokenContract;

if (!fromIsNative) {
// case preHook, we need to check balance / allowance instead of fromToken
// to avoid changing the entire approach, we only inject the address on the contract instance for on chain validation
// need to be considered that fundToken is unknown and we probably do not support
fromTokenContract = ethersAdapter.contract(
_fromToken.address,
preHook ? preHook.fundToken : _fromToken.address,
erc20Abi,
signer || fromProvider,
);
Expand Down

0 comments on commit 28c5eab

Please sign in to comment.