-
Notifications
You must be signed in to change notification settings - Fork 375
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
[Wallet] Wallet can switch between hosted and local node #1419
Merged
Merged
Changes from 65 commits
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
1242a1f
Always write key to disk to allow to switch into zero sync mode
annakaz 1e7fa99
Add celo lite settings component
annakaz 1eaf14d
Add celo lite settings link
annakaz 5f1443e
Update state to keep track of sync mode, account in web3 keystore
annakaz d170908
Make celo lite settings screen responsive
annakaz e49e37a
WIP switching
annakaz 6e0f4bf
Merge branch 'master' into annakaz/zero-sync-option
annakaz 304b8c2
Fixing redux saga logic
annakaz a017e5b
Get sagas working
annakaz b34d55d
Move zeroSync logic out, refactor assignAccountFromPrivateKey
annakaz b0dc9b3
Moving more logic to web3/zeroSync
annakaz 2f9dc16
Merge branch 'master' into annakaz/zero-sync-option
annakaz d46e1cf
Yield select whenever zeroSync mode needed
annakaz 8f7db2c
Switching between providers
annakaz eafce36
check sync mode via web3 provider
annakaz e12c0cf
switch web3 provider when necessary
annakaz dae95b3
Default to starting in full sync mode
annakaz a9a3dde
Revert changes to functions using sendTransaction
annakaz cfcd1fb
Handle isZeroSync mode using web3 object
annakaz 15d7653
Ensure private key exists/web3 keystore exists for switch
annakaz b71ad0c
Switch to always reading zeroSync state from web3
annakaz efcfcee
Merge branch 'master' into annakaz/zero-sync-toggle
annakaz 76abb84
Stop storing account in web3 keystore separately
annakaz 9b44218
Remove zerosync logic
annakaz cfa940c
Remove zerosync file
annakaz bb210fa
Revert logic changes to save for diffrerent pr
annakaz 8e5d46b
Remove some unused styles
annakaz d79988c
Comment out celoLite settings for now
annakaz 8735f97
Add spanish translations, a couple comments
annakaz 332e24e
Add jest tests
annakaz 03cfa6f
Add Licenses test
annakaz 1d33eb9
Update spanish translations
annakaz 6843c4e
Fix linting error
annakaz e8a529d
Merge branch 'master' into annakaz/zero-sync-toggle
annakaz 8a5a917
Merge branch 'master' into annakaz/zero-sync-toggle
annakaz 456f72f
Merge branch 'master' into annakaz/zero-sync-toggle
annakaz 3c448b4
Switch to generator fns in verification
annakaz 93c15b3
Field yield all
annakaz b0b7401
Move invite to saga (working and tested on integration)
annakaz 74c73da
WIP turing send to generators, not working
annakaz 8e18032
Finish switch to generators
annakaz 3beff76
Use zeroSync state throughout
annakaz 6082761
Finish switch over to only using state, excet in intialization web3
annakaz cc32812
WIP getting zerosync to actually turn on
annakaz 3adf667
Merge branch 'master' into annakaz/zero-sync-toggle-logic
annakaz 72b46ba
Get switch from geth to zerosync working (tested w exchange, only sam…
annakaz e6b1139
Add back in zeroSync check in addLocalAccount
annakaz 0d306a8
Turn zeroSync on/off now working in same session (geth always running)
annakaz 182d1c8
Add sync mode checker to initial app load. Tested going from geth to …
annakaz ebbe915
Rename to initiallyZeroSync mode, use put in toggle saga
annakaz a0fa526
App refresh for either mode
annakaz bd92712
web3/saga cleanup
annakaz df3d392
Store zeroSync param separate from sync mode
annakaz f0fa40d
Merge branch 'master' into annakaz/zero-sync-toggle-logic
annakaz ff7261a
Update env files with zeroSync mode
annakaz 9cd47ca
Update accounts snapshot
annakaz 376920d
Clean up for review
annakaz 9c7fd33
Make changes based on review
annakaz f2bf8c1
Merge branch 'master' into annakaz/zero-sync-toggle-logic
annakaz 5bc42c5
Remove unnecessary yield puts causing loop"
annakaz 30fa17d
Turn geth off, fix tests, use back button in header
annakaz 59ae0f6
Make zeroSyncInitially false for all env files
annakaz 9fbb5c8
Merge branch 'master' into annakaz/zero-sync-toggle-logic
annakaz 5faf4b6
Wait for web3 to sync after switching to geth
annakaz d45f1e6
PR cleanup
annakaz a615862
Merge branch 'master' into annakaz/zero-sync-toggle-logic
annakaz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,8 +1,10 @@ | ||
ENVIRONMENT=local | ||
DEFAULT_TESTNET=integration | ||
# -1 == ZeroSync, 5 == Ultralight, see src/geth/consts.ts for more info | ||
DEFAULT_TESTNET=alfajores | ||
# If ZERO_SYNC_ENABLED_INITIALLY, local geth will not run initially. | ||
# If toggled on, it will use DEFAULT_SYNC_MODE. See src/geth/consts.ts for more info | ||
ZERO_SYNC_ENABLED_INITIALLY=false | ||
DEFAULT_SYNC_MODE=5 | ||
DEV_SETTINGS_ACTIVE_INITIALLY=true | ||
FIREBASE_ENABLED=true | ||
SECRETS_KEY=debug | ||
SHOW_TESTNET_BANNER=true | ||
SHOW_TESTNET_BANNER=true |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Not sure why my pre commit hook reformats this file
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.
yeah it did the same for me, i had to manually revert it. But it's nbd feel free to leave in this PR