Skip to content

feat(tickets): add a basic (barren) React tickets list page - #5181

Open
wescopeland wants to merge 3 commits into
stack/tickets/3-index-backendfrom
stack/tickets/4-tickets2-page
Open

feat(tickets): add a basic (barren) React tickets list page#5181
wescopeland wants to merge 3 commits into
stack/tickets/3-index-backendfrom
stack/tickets/4-tickets2-page

Conversation

@wescopeland

Copy link
Copy Markdown
Member

This PR builds the initial React tickets index page. There is a lot missing from this page - intentionally - to narrow this branch's scope. Notably, there are no UI controls of any kind on the page. Mobile is also lacking. This branch's changes are foundational work for the UI, just to get something workable on the screen that can be built on top of.

http://localhost:64000/tickets2
http://localhost:64000/tickets2?filter[status]=all
http://localhost:64000/tickets2?page[number]=2

The UI here already has slight differences from our conventional datatables, even though it very much is a datatable:

  • Rows themselves are clickable, leading to the ticket details page.
  • Only the ticket ID has a link color, even though the other entities (achievement, leaderboard, game, users) are clickable too. This is to maintain the data hierarchy on the page - if everything gets a link color the hierarchy gets extremely muddied.
  • Based on the screen width, text from one column may bleed into text on another column. This is a known issue and will be fixed in a subsequent PR.

Status is represented with an a11y-aware Linear-like glyph:

Screenshot 2026-08-19 at 5 35 43 PM Screenshot 2026-08-19 at 5 36 10 PM Screenshot 2026-08-19 at 5 34 59 PM Screenshot 2026-08-19 at 5 35 26 PM

Eventually, I'd like to see these statuses not only apply to tickets which a dev needs to jump on, but also general "issues" like UWC reports, etc. Eventually, the "Ticket Manager" may become an "Issue Manager", and users will have an "inbox" that shows them the current status of all their issues.

Screenshot 2026-08-19 at 5 29 43 PM

@wescopeland
wescopeland requested a review from a team August 19, 2026 21:40
@greptile-apps

greptile-apps Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds the initial authenticated React ticket-list page.

  • Builds and serializes ticket-list props.
  • Renders responsive ticket rows, links, status glyphs, and empty states.
  • Adds backend and frontend coverage.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Reviews (2): Last reviewed commit: "test: add coverage" | Re-trigger Greptile

Comment thread app/Platform/Controllers/TicketController.php

@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.

It's hard to tell whether functionality is intentionally or unintentionally missing (like tooltips).

The basic grid functionality and links all seem to work. I couldn't figure out how to make the status grouping headers appear.

Did not look at code.

Comment thread resources/js/features/tickets/utils/getTicketStateLabel.ts
@wescopeland
wescopeland force-pushed the stack/tickets/4-tickets2-page branch from 0efed77 to 6f66a28 Compare August 26, 2026 23:16
@wescopeland
wescopeland force-pushed the stack/tickets/4-tickets2-page branch from 6f66a28 to aa96064 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