47ng / nuqs

Type-safe search params state manager for React frameworks - Like useState, but stored in the URL query string.
https://nuqs.47ng.com
MIT License
4.86k stars 104 forks source link

doc: Add limits page to docs #739

Closed niklasbec closed 2 weeks ago

niklasbec commented 2 weeks ago

I added a new documentation page that informs users about some limitations when using nuqs. If you would like to have anything changed/added, let me know.

Closes #731.

image

vercel[bot] commented 2 weeks ago

Someone is attempting to deploy a commit to the 47ng Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] commented 2 weeks ago

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 Nov 5, 2024 9:11pm
niklasbec commented 2 weeks ago

@franky47 Thanks for the feedback, did some updates. Let me know what you think.

By the way, do you think it is worth to display a warning in development if the limit of 2,000 is exceeded? If so I could work on that.

niklasbec commented 2 weeks ago

LGTM, there might be some copy tweaks to do but I'll deal with that later, thanks!

Regarding the warning for max URL size, that sounds like a good idea, feel free to open a separate PR for this. You could log an error that points back to this documentation page actually, using the same error codes (see errors.ts, 414 would be perfect for this).

Yeah something like that, will give it a go the next few days.

github-actions[bot] commented 1 week ago

:tada: This PR is included in version 2.1.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket: