-
-
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
Unable to Speedup/Cancel legacy transactions #4972
Conversation
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. |
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.
Since you’re already checking with an optional in that if
statement, you can remove the optionals inside the if
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
Tested..... Speed Up: PASS |
Description
Non-EIP1559 transactions breaks when a user tries to speed up transaction. This test was done using BNB network and also applies to Optimism.
Screenshots/Recordings
https://recordit.co/1sIbNiNrfA
Issue
Progresses https://github.com/MetaMask/mobile-planning/issues/380
Checklist