-
-
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
Custom Gas + Data hotfixes #1687
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.
LGTM
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.
I see the data, however, it's not scrollable to show me all of the data as compared to extension; seen here = https://recordit.co/7WyO7lKu4u
@ibrahimtaveras00 should be good |
…nsactionReviewFeeCard
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.
I have my currency set to fiat, and upon clicking on deploy contract on https://metamask.github.io/test-dapp/ site to test if data is scrollable the app crashed; seen here = http://recordit.co/UwSOKhH9nb
although it does look like the deeplink issue was fixed when fiat is set = http://recordit.co/AXjBl8Vud7
Also, when primary currency is set to native, I see this warning when tapping on deploy contract
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.
All issues have been resolved, QA Passed
* CustomGas: fix crash when navigating back to advanced gas during error * TransactionReviewData: fix data not displaying * fix snapshit test * TransactionReviewData: fix data not scrollable * snapsho * SendFlow: fix total fiat to be text component when passed down to TransactionReviewFeeCard * snapshotsssss baby * TransactionReviewInformation: fix totalFiat is now a Text component
Description
A couple of bug fixes
Checklist
Issue
Fixes https://trello.com/c/ESDSkNmz/171-data-not-showed-on-approve-modal
Fixes https://trello.com/c/6cI8Cm7K/177-deeplinking-bugs
Fixes https://trello.com/c/0rESeMgP/173-changing-to-gas-selectors-from-gas-input-and-back-to-gas-input-with-a-gas-price-error-crashes-the-app