-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat: add account syncing #14291
feat: add account syncing #14291
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. |
|
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #14291 +/- ##
==========================================
+ Coverage 66.29% 66.56% +0.26%
==========================================
Files 2248 2251 +3
Lines 48037 48144 +107
Branches 6766 6782 +16
==========================================
+ Hits 31848 32047 +199
+ Misses 14172 14076 -96
- Partials 2017 2021 +4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
|
|
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.
Validated e2e and locally
Description
This PR adds account syncing to the mobile client.
This feature, that is currently live on extension, syncs EVM accounts & their names between devices.
With this PR, account syncing will effectively work between extension and mobile!
This PR:
importAdditionalAccounts
so that it occurs after every kind of onboarding. This is important for account syncing as we rely on this to be complete before actually dispatching account syncing. This is done to prevent race conditions that adds accounts at the same time. The same logic is currently applied on extension with the_addAccountsWithBalance
method.useAccountSyncing
hookuseIdentityEffects
so that it reactively dispatches account syncing when all pre-requisites are met.identity
Bitrise iOS and Android workflows, and adds new E2E tests for account syncingRelated issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist