Skip to content
This repository has been archived by the owner on Apr 25, 2024. It is now read-only.

Update TODO comment with correction #144

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/entities/protocols/uniswap.ts
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ export class UniswapTrade implements Command {

// If the input currency is the native currency, we need to wrap it with the router as the recipient
if (this.trade.inputAmount.currency.isNative) {
// TODO: optimize if only one v2 pool we can directly send this to the pool
// leave the WETH in the router, as this is cheapest gas-wise than sending it straight to a V2 Pair
planner.addCommand(CommandType.WRAP_ETH, [
ROUTER_AS_RECIPIENT,
this.trade.maximumAmountIn(this.options.slippageTolerance).quotient.toString(),
Expand Down