-
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
2031-In-health-page---on-first-load-we-hide-passing-ones-by-default-However-the-Hide-Passing-toggle-is-switched-off #2040
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for flanksource-demo-stable ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for clerk-saas-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for goofy-euclid-75956c ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
8113c5e
to
b8bc226
Compare
b8bc226
to
3355756
Compare
@mainawycliffe this doesn't seem to work - The UI is taking it value from query params and the query is using local storage Can we not just use atomWithStorage, I don't think hidePassing should update the URL state |
@moshloop we can, but I thought when sharing a link, you would want to share exactly what you looking at in terms of state? |
3355756
to
c687994
Compare
@mainawycliffe - This doesn't seem to work - the toggle reflects the local storage state but the view does not |
c687994
to
0ef0515
Compare
0ef0515
to
59454ff
Compare
Fixes #2031