-
-
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
Refactor SendTransaction Component #4835
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. |
app/components/UI/TransactionReview/TransactionReviewEIP1559Update/index.tsx
Outdated
Show resolved
Hide resolved
app/components/UI/TransactionReview/TransactionReviewEIP1559Update/index.tsx
Outdated
Show resolved
Hide resolved
4f26475
to
2ccddff
Compare
2ccddff
to
6918633
Compare
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.
Minor comments
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
6918633
to
53853f6
Compare
cb50e7c
to
62752d2
Compare
53853f6
to
825b222
Compare
48bd3ff
to
2c30085
Compare
@blackdevelopa not able to send on Binance testnet. Getting JSON-RPC error... |
Send refactor looks good on the following testnets....
|
resolved. http://recordit.co/1ZkJUCgmb0 |
Send is good on BNB testnet now. |
Description
There is very similar code spread across 5 components. This PR removes the eip1559 gas polling logic away from the send flow (confirm screen). The TransactionReview and EditGasFee screens is able to calculate the gas transaction cost. This logic is abstracted away in the gas polling file. The changes in this PR shouldn't modify existing behaviour when QA'd.
Progresses
https://github.com/MetaMask/mobile-planning/issues/272
Screenshots/Recordings
Screen.Recording.2022-08-10.at.20.15.12.mov
Issue
Progresses https://github.com/metamask/mobile-planning/issues/269
Checklist