Skip to content

ref(explore): Migrate context.tsx to nuqs#115435

Open
ryan953 wants to merge 1 commit into
masterfrom
ryan953/migrate-context-nuqs
Open

ref(explore): Migrate context.tsx to nuqs#115435
ryan953 wants to merge 1 commit into
masterfrom
ryan953/migrate-context-nuqs

Conversation

@ryan953
Copy link
Copy Markdown
Member

@ryan953 ryan953 commented May 12, 2026

Summary

  • Replaces updateNullableLocation with nuqs useQueryStates in useSetQueryParamsSavedQuery
  • Removes dependencies on useLocation, useNavigate, and updateNullableLocation

Test plan

  • Lint passes
  • Verify saving query params (id/title) works correctly in explore

…Location to nuqs

Replace the manual location/navigate approach with nuqs useQueryStates.
This removes the dependency on useLocation, useNavigate, and updateNullableLocation
in favor of nuqs parseAsString parsers for the id and title query params.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ryan953 ryan953 requested a review from a team as a code owner May 12, 2026 21:28
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.50%

@ryan953 ryan953 requested a review from a team May 12, 2026 21:49
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.

1 participant