Skip to content

ref(search): read and update search bar URL params with nuqs - #129648

Open
JoshuaKGoldberg wants to merge 1 commit into
joshgoldberg/de-1793-nuqs-test-adapterfrom
joshgoldberg/de-1796-nuqs-search-bars
Open

JoshuaKGoldberg wants to merge 1 commit into
joshgoldberg/de-1793-nuqs-test-adapterfrom
joshgoldberg/de-1796-nuqs-search-bars

Conversation

@JoshuaKGoldberg

@JoshuaKGoldberg JoshuaKGoldberg commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

The alerts and monitors list search bars and the Seer runs and autofix issues demo pages now read and update their search query through nuqs, finishing the search bars that were waiting on the testing adapter fix.

This is +164/-0 for tests, so it's kind of +27/-44. 🔪

Closes DE-1796.

The alerts and monitors list search bars and the Seer runs and autofix
issues demo pages now read their query and update it, dropping the
cursor, through nuqs instead of the router location.

Refs DE-1796.
@linear-code

linear-code Bot commented Nov 3, 2026

Copy link
Copy Markdown

DE-1796

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 3, 2026
@JoshuaKGoldberg

Copy link
Copy Markdown
Member Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit d0ed997. Configure here.

@JoshuaKGoldberg
JoshuaKGoldberg marked this pull request as ready for review November 3, 2026 16:22
@JoshuaKGoldberg
JoshuaKGoldberg requested a review from a team as a code owner November 3, 2026 16:22

This branch was successfully deployed

1 active deployment
Preview — d0ed9978 Deployed Nov 3, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants