Skip to content

doc: add tanstack multiple paginations example #902

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

Open
wants to merge 1 commit into
base: next
Choose a base branch
from

Conversation

stychu
Copy link

@stychu stychu commented Feb 7, 2025

Update the documentation with a pagination example how to use multiple pagination/tables with nuqs and search params.

Copy link

vercel bot commented Feb 7, 2025

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

A member of the Team first needs to authorize it.

@franky47
Copy link
Member

franky47 commented Feb 7, 2025

Thanks! There are a few other PRs related to TanStack Table parsers & docs, I'll have to see how to best combine them.

Regarding "namespacing" keys in the URL to support multiple instances of a component without state overlap, I was thinking of adding a prefix option for the urlKeys feature, that would still return the state names as declared, but add that prefix automatically in the URL, what do you think?

@stychu
Copy link
Author

stychu commented Feb 10, 2025

Regarding "namespacing" keys in the URL to support multiple instances of a component without state overlap, I was >thinking of adding a prefix option for the urlKeys feature, that would still return the state names as declared, but add that >prefix automatically in the URL, what do you think?

Sure that will also do !
But for our use case if we go with two separate params per pagination the url will become a mess quickly as we have 3-10 tables per url so with array param its much cleaner.

@junwen-k
Copy link

I am the author for the related PR #836 :) Feel free to let me know if there's anything that I can help with adding Tanstack Table as community examples. @franky47

@franky47
Copy link
Member

@junwen-k thanks, I've had your PR in my review list for way too long (sorry about that), I need to take some time at some point to do a good pass on all docs related PRs.

josealfalf

This comment was marked as spam.

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

Successfully merging this pull request may close these issues.

4 participants