Skip to content
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

Show/Hide Fiat on Testnets based on User Preference #6153

Merged
merged 19 commits into from
Feb 26, 2019

Conversation

chikeichan
Copy link
Contributor

@chikeichan chikeichan commented Feb 14, 2019

showhide

  • Show/Hide Fiat based on network type for sidebar, transaction-list, transaction-list-item balance display
  • Show/Hide Fiat based on network type for account balance and currency-input on send screen
  • Show/Hide Fiat based on network type for gas selectors on send screen
  • Show/Hide Fiat based on network type for currency-input on send token screen
  • Show/Hide Fiat based on network type for gas selectors on send token screen
  • Show/Hide Fiat based on network type for all balances on confirm screen
  • Show/Hide Fiat based on network type for all balances on confirm token screen
  • Test Show/Hide in Contract Deploy
  • Test Show/Hide in non-send actions (contract calls, sign message, etc)
  • Add New Preference toggle
  • Add Unit Tests

@chikeichan chikeichan added the DO-NOT-MERGE Pull requests that should not be merged label Feb 14, 2019
@chikeichan chikeichan changed the title [DO NOT MERGE] Show/Hide Fiat on Testnets based on User Preference Show/Hide Fiat on Testnets based on User Preference Feb 15, 2019
@chikeichan chikeichan removed the DO-NOT-MERGE Pull requests that should not be merged label Feb 15, 2019
@chikeichan
Copy link
Contributor Author

  • cherrypicked e2e test fix from @danjm

danjm
danjm previously approved these changes Feb 25, 2019
@danjm danjm force-pushed the hide-fiat-testnets branch from dcc61a0 to 2d732aa Compare February 26, 2019 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants