-
Notifications
You must be signed in to change notification settings - Fork 42
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
Refactor selecting network initially to avoid sync timing issues #1343
Conversation
Deploying with
|
Latest commit: |
d216857
|
Status: | ✅ Deploy successful! |
Preview URL: | https://44d46b4a.oasis-wallet.pages.dev |
Branch Preview URL: | https://lw-sync-time.oasis-wallet.pages.dev |
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
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 but you have to bump some snapshots
Slightly refactored default network type, so that tests with configureAppStore can stay simple as they were |
Reverts part of c125984 and re-fixes #975 (comment) Instead of waiting for tabs to be synced before selecting network initially: - initialize selecting network without syncing that action - using takeLatest cancel initialization task if tabs sync during selectNetwork
Codecov Report
@@ Coverage Diff @@
## master #1343 +/- ##
==========================================
- Coverage 84.77% 84.15% -0.63%
==========================================
Files 142 142
Lines 3612 3628 +16
Branches 661 667 +6
==========================================
- Hits 3062 3053 -9
- Misses 550 575 +25
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Related to #792 (comment)
Reverts part of c125984 and re-fixes #975 (comment)
Instead of waiting for tabs to be synced before selecting network initially: