Skip to content

Fix 9988 - Don't allow more than 15% slippage#9991

Merged
darkwing merged 1 commit intoMetaMask:developfrom
darkwing:max-slippage-9988
Dec 7, 2020
Merged

Fix 9988 - Don't allow more than 15% slippage#9991
darkwing merged 1 commit intoMetaMask:developfrom
darkwing:max-slippage-9988

Conversation

@darkwing
Copy link
Copy Markdown
Contributor

@darkwing darkwing commented Dec 3, 2020

Fixes: #9988

Explanation: Per #9988, we want to disable slippage over 15%. We're also changing the label and tooltip to use the word "tolerance".

Manual testing steps:

  • Start a swap
  • Set a custom slippage of "1", see no error and button enabled
  • Set a custom slippage of "11", see warning and button enabled
  • Set a custom slippage of "111", see error and button disabled

@darkwing darkwing requested a review from danjm December 3, 2020 17:59
@darkwing darkwing requested a review from a team as a code owner December 3, 2020 17:59
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 3, 2020

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

Copy link
Copy Markdown
Contributor

@danjm danjm left a comment

Choose a reason for hiding this comment

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

LGTM

@darkwing darkwing merged commit 56f80ae into MetaMask:develop Dec 7, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Dec 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Block users from setting high Slippage

2 participants