-
Notifications
You must be signed in to change notification settings - Fork 22
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
fix: resolve blip of UI before notices redirect, refactor NProgress loader #993
fix: resolve blip of UI before notices redirect, refactor NProgress loader #993
Conversation
@@ -56,6 +37,10 @@ const Root = () => { | |||
); | |||
} | |||
|
|||
if (!isAppDataHydrated) { |
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.
[inform] Accounts for both user profile hydration and whether there's a notices redirect.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feat/react-query-route-loaders #993 +/- ##
==================================================================
+ Coverage 81.25% 81.40% +0.14%
==================================================================
Files 371 374 +3
Lines 7773 7792 +19
Branches 1888 1893 +5
==================================================================
+ Hits 6316 6343 +27
+ Misses 1401 1393 -8
Partials 56 56 ☔ View full report in Codecov by Sentry. |
bc2a7a0
into
feat/react-query-route-loaders
For all changes
Only if submitting a visual change