Skip to content

[Request Invoicing] Remove pagination so that filtering and sorting work. Keep tanstack query and caching #335

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
MantisClone opened this issue Feb 4, 2025 · 0 comments · May be fixed by #341

Comments

@MantisClone
Copy link
Member

MantisClone commented Feb 4, 2025

Problem

Paginated decryption is not compatible with filtering and sorting. Users expect global filtering and sorting, which requires all of their requests to be decrypted.

Solution

Remove pagination so that filtering and sorting work as expected. Keep tanstask query and the caching it provides.

@MantisClone MantisClone changed the title [Request Invoicing] Remove pagination so that filtering and sorting work [Request Invoicing] Remove pagination so that filtering and sorting work. Keep tanstack query and caching Feb 4, 2025
@MantisClone MantisClone moved this from 🆕 New to 🔖 Sprint Backlog in Request Network Tech Backlog Feb 4, 2025
@rodrigopavezi rodrigopavezi self-assigned this Feb 5, 2025
@MantisClone MantisClone added this to the Request Invoicing V3 milestone Feb 5, 2025
@sstefdev sstefdev assigned sstefdev and unassigned rodrigopavezi Feb 19, 2025
@sstefdev sstefdev linked a pull request Mar 4, 2025 that will close this issue
@sstefdev sstefdev moved this from 🔖 Sprint Backlog to 👀 In Review in Request Network Tech Backlog Mar 4, 2025
@sstefdev sstefdev linked a pull request Mar 4, 2025 that will close this issue
@MantisClone MantisClone moved this from 👀 In Review to 🎫 Backlog in Request Network Tech Backlog Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🎫 Backlog
Development

Successfully merging a pull request may close this issue.

3 participants