Skip to content

feat(tickets): prepare BuildTicketListAction for React tickets list - #5176

Open
wescopeland wants to merge 1 commit into
stack/tickets/2-service-refactorfrom
stack/tickets/3-index-backend
Open

feat(tickets): prepare BuildTicketListAction for React tickets list#5176
wescopeland wants to merge 1 commit into
stack/tickets/2-service-refactorfrom
stack/tickets/3-index-backend

Conversation

@wescopeland

Copy link
Copy Markdown
Member

This is a native stacked PR. #5173 should be reviewed first. Some of the changes in this PR may contextually make more sense after additional stacked PRs are opened. The entire React tickets index feature is already built on my local, but I am splitting it up into multiple PRs so it can be viewed through less-overwhelming chunks.


This PR prepares a BuildTicketListAction to ultimately support a React/Inertia-powered tickets index page. There will be a number of PRs required to facilitate this migration in order to cover the 7 existing pages.

BuildTicketListAction is hooked into TicketApiController.php and a ticket-list.blade.php demo page (to prove it works). This demo page is accessible only by Root users and will be deleted in a subsequent PR.

There are three behavior differences in the backend contract for this diff:

  • filter[status] has a new "request" value.
  • resolved-by now defaults to Resolved and Closed.
  • awaiting-reporter now honors an explicit status value instead of clobbering it.

The legacy Blade pages are unaffected by these changes.

http://localhost:64000/demo/ticket-list

Screenshot 2026-08-17 at 5 43 31 PM

@wescopeland
wescopeland requested a review from a team August 17, 2026 21:51
@greptile-apps

greptile-apps Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds a reusable ticket-list builder and authenticated JSON endpoint.

  • Adds scoped filtering, sorting, pagination, and state counts.
  • Adds ticket-list DTOs, enums, generated types, and tests.
  • Adds a Root-only demo page.

Confidence Score: 5/5

The PR appears safe to merge.

The new API preserves documented legacy scope behavior, and the intentional resolved-by change is tested.

Reviews (1): Last reviewed commit: "feat(tickets): prepare BuildTicketListAc..." | Re-trigger Greptile

@wescopeland
wescopeland force-pushed the stack/tickets/3-index-backend branch from af7c9d1 to 8eaa80d Compare August 26, 2026 23:16
@wescopeland
wescopeland force-pushed the stack/tickets/3-index-backend branch from 8eaa80d to d118341 Compare August 31, 2026 20:22
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