Skip to content

Ensure users reacknowledge price slippage risks when it changes#10376

Merged
darkwing merged 1 commit intoMetaMask:developfrom
darkwing:risk-bucket-acknowledgement-change
Feb 5, 2021
Merged

Ensure users reacknowledge price slippage risks when it changes#10376
darkwing merged 1 commit intoMetaMask:developfrom
darkwing:risk-bucket-acknowledgement-change

Conversation

@darkwing
Copy link
Copy Markdown
Contributor

@darkwing darkwing commented Feb 4, 2021

Fixes the case where the user agrees to slippage changes but the next quote cycle changes that slippage risk from medium to high.

@darkwing darkwing requested a review from danjm February 4, 2021 19:51
@darkwing darkwing requested a review from a team as a code owner February 4, 2021 19:51
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 4, 2021

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.

danjm
danjm previously approved these changes Feb 5, 2021
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 force-pushed the risk-bucket-acknowledgement-change branch from 14587fc to 80cb9ee Compare February 5, 2021 18:55
@darkwing darkwing merged commit 531c35b into MetaMask:develop Feb 5, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Feb 5, 2021
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.

2 participants