-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bugfix/speedup transactions #1684
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Speeding up and canceling work fine on regular send flow 👍
Issue 1:
I did a token swap on uniswap, however, when I tapped on the speed up button nothing happened and I got the following warning:
{"code": -32000, "message": "insufficient funds for gas * price + value"}
Seen here = https://recordit.co/DCyD54MJTm
Issue 2:
I believe the button's color should be updated to primary-blue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 👍
added a change for that issue on MetaMask/core#249 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix looks good, QA Passed 👍
* add padding * create util * new gaba * buttons color * confirmButtonMode={'confirm'}
Description
The validation method was wrong. I also moved it to an method in utils.
Checklist
Issue
Resolves #???