Skip to content
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: Allow parser-level options in useQueryStates #621

Merged
merged 3 commits into from
Sep 1, 2024

Conversation

franky47
Copy link
Member

@franky47 franky47 commented Sep 1, 2024

Order of precedence (first non-nullish wins):

  • call level
  • parser level
  • hook-global level (otherwise default).

Also fixes a bug with clearOnDefault in useQueryState where a true top-level value could not be overriden by a call-level false value.

Closes #618.

Order of precedence (first non-nullish wins):
- call level
- parser level
- hook-global level (otherwise default).

Also fixes a bug with `clearOnDefault` in useQueryState where
a `true` top-level value could not be overriden by a call-level `false` value.

Closes #618.
Copy link

vercel bot commented Sep 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nuqs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 1, 2024 8:14pm

@franky47 franky47 enabled auto-merge (squash) September 1, 2024 20:13
@franky47 franky47 disabled auto-merge September 1, 2024 20:13
@franky47 franky47 enabled auto-merge (squash) September 1, 2024 20:13
@franky47 franky47 merged commit 099ceb3 into next Sep 1, 2024
16 checks passed
@franky47 franky47 deleted the fix/usequerystates-options branch September 1, 2024 20:14
Copy link

github-actions bot commented Sep 2, 2024

🎉 This PR is included in version 1.19.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

github-actions bot commented Sep 2, 2024

🎉 This PR is included in version 1.19.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

clearOnDefault doesn't work in useQueryStates
1 participant