-
-
Notifications
You must be signed in to change notification settings - Fork 405
Fix tools filter checkbox state persistence on refresh #2058
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
base: main
Are you sure you want to change the base?
Conversation
|
Hi @RitoG09! Thanks a lot for your contribution! I noticed that the following required information is missing or incomplete: kind of change description Please update the PR description to include this information. You can find placeholders in the PR template for these items. Thanks a lot! |
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2058 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 30 30
Lines 633 633
Branches 196 196
=========================================
Hits 633 633 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@Utkarsh-123github I have rectified it, can u review it once? |
Utkarsh-123github
left a comment
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.
Hi @RitoG09
I ran your fix preview, and the issue still persists:
Recording.2026-01-07.133754.mp4
@Utkarsh-123github , I made some changes for this issue you showed which is running correctly in my localhost but I'm unable to run this in production. Can you guide me how to run this on production? |
What kind of change does this PR introduce?
Bugfix
Issue Number:
Screenshots/videos:
Screen.Recording.2026-01-01.175829.mp4
If relevant, did you update the documentation?
Not required.
Summary
Fixes an issue where the “Show obsolete” and “Support Bowtie” filters did not persist their checked state on page refresh. The filter state is now correctly hydrated from URL query parameters, and the checkbox UI updates immediately on interaction.
Does this PR introduce a breaking change?
No
Checklist
Please ensure the following tasks are completed before submitting this pull request.