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

App settings #148

Merged
merged 14 commits into from
Nov 14, 2018
Merged

App settings #148

merged 14 commits into from
Nov 14, 2018

Conversation

estebanmino
Copy link
Contributor

Description

This PR adds a new app settings view with the following:

  1. Change currency
  2. Change language
  3. Add custom RPC url
  4. Reveal seed words, after password validation
  5. Clear transaction history with Reset Account button

settings

For this GIF ganache was running on background to show the ability to add custom RPC url.

Component RevealPrivateCredential from MetaMask/website#147 was used for revealing seed words. Tests are included there, probably we want to merge that PR first.

WIP

Waiting for MetaMask/core#20
Language and currency go back to default when app is turned off.

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

Issue

Resolves #65

@estebanmino
Copy link
Contributor Author

estebanmino commented Oct 24, 2018

UPDATE

update

Added:

  1. Delete custom RPC URL (waiting for Update Preferences state core#21)
  2. Persist preferred language when app shutted down
  3. Persist currentCurrency when app shutted down (waiting for Maintain activeCurrency in CurrencyRateController state core#22)
  4. Reload app when preference language changes

GABA PRs shouldn't be blockers for this PR to merge, when those PRs are merged we only need to update the version.

@estebanmino estebanmino changed the title [WIP] App settings App settings Oct 24, 2018
@brunobar79 brunobar79 self-requested a review November 7, 2018 19:32
Copy link
Contributor

@brunobar79 brunobar79 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@estebanmino This looks great to me!

@brunobar79 brunobar79 mentioned this pull request Nov 13, 2018
@estebanmino estebanmino merged commit 9fd6178 into master Nov 14, 2018
This was referenced Nov 20, 2018
@estebanmino estebanmino deleted the app-settings branch January 8, 2019 18:24
rickycodes pushed a commit that referenced this pull request Jan 31, 2022
* new branch app settings with languages selection and reveal seed words working

* app settings change current conversion

* app settings reset account with new action modal component

* changes to app settings view and add custom rpc url

* rpc url validation

* remove rpc target WIP

* delete custom rpc url

* persist language and change navbar according to language

* update snapshot

* reload app when language is changed

* update GABA version

* eth-block-tracker to v3.0.1 and use of react-native-modal

* update unit tests
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.

2 participants