Skip to content

feat(tickets): add client-side pagination to tickets2 - #5182

Merged
wescopeland merged 3 commits into
stack/tickets/4-tickets2-pagefrom
stack/tickets/5-client-pagination
Sep 4, 2026
Merged

feat(tickets): add client-side pagination to tickets2#5182
wescopeland merged 3 commits into
stack/tickets/4-tickets2-pagefrom
stack/tickets/5-client-pagination

Conversation

@wescopeland

Copy link
Copy Markdown
Member

This PR primarily adds client-side pagination to the tickets2 page:

Screenshot 2026-08-20 at 6 55 41 PM

Mobile rows have also been compressed a bit, but those are still WIP.

@wescopeland
wescopeland requested a review from a team August 20, 2026 22:59
@greptile-apps

greptile-apps Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds reusable client-side pagination and ticket-list query prefetching.

  • Synchronizes ticket pagination with URL parameters.
  • Reuses pagination controls in game lists.
  • Compresses mobile ticket rows.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Reviews (3): Last reviewed commit: "fix: a11y" | Re-trigger Greptile

Comment thread resources/js/features/tickets/hooks/useTicketListTableSync.ts

@Jamiras Jamiras left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems functional. Did not review code.

@wescopeland
wescopeland force-pushed the stack/tickets/5-client-pagination branch from 8bc8741 to f872da1 Compare August 26, 2026 23:16
@wescopeland
wescopeland force-pushed the stack/tickets/5-client-pagination branch from f872da1 to 5d68694 Compare August 31, 2026 20:22
@wescopeland
wescopeland force-pushed the stack/tickets/5-client-pagination branch from 5d68694 to ef68575 Compare September 4, 2026 17:41
@wescopeland
wescopeland merged commit 7dcea9d into master Sep 4, 2026
24 of 33 checks passed
@wescopeland
wescopeland deleted the stack/tickets/5-client-pagination branch September 4, 2026 18:07
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.

2 participants