Skip to content

fixed the min amount with slippage#164

Open
fakoredeDamilola wants to merge 1 commit intodevelopfrom
bug/slippage-min
Open

fixed the min amount with slippage#164
fakoredeDamilola wants to merge 1 commit intodevelopfrom
bug/slippage-min

Conversation

@fakoredeDamilola
Copy link
Collaborator

What does this PR do?

  • this fixed the minAmountOut for token A ND B by calculating the value from the slippage

What has been completed?

  • [ ]
  • [ ]

How should the changes be manually tested?

Any Background context or information you want to provide?

Demo Video and screenshots?

Copy link
Contributor

@kayroy247 kayroy247 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kindly implement the feedback.

const amountBMin = Web3.utils.toWei(minimumAmountToReceive(toValue).toString())
// const amountAMin = Web3.utils.toWei((fromValue * 0.8).toString());
// const amountBMin = Web3.utils.toWei((toValue * 0.8).toString());
console.log({amountAMin,amountBMin})
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kindly remove the console log statements and the commented codes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants