Skip to content

feat(sig): add modifying report state via inbox#1513

Open
oliverb123 wants to merge 2 commits intomainfrom
sig/report-actions
Open

feat(sig): add modifying report state via inbox#1513
oliverb123 wants to merge 2 commits intomainfrom
sig/report-actions

Conversation

@oliverb123
Copy link
Copy Markdown
Contributor

@oliverb123 oliverb123 commented Apr 7, 2026

Problem

We want to be able to snooze, suppress, delete reports via the PHC inbox. We also want to be able to reingest them, although this is gated behind 1: being a staff user and 2: running a dev build.

This PR also contains some futzing with the keyboard handling, because it wasn't as good as I wanted it to be

image.png

Copy link
Copy Markdown
Contributor Author

oliverb123 commented Apr 7, 2026

@oliverb123 oliverb123 marked this pull request as ready for review April 7, 2026 11:41
@oliverb123 oliverb123 requested a review from a team April 7, 2026 11:42
@oliverb123 oliverb123 force-pushed the sig/report-actions branch from 48e4461 to 2fcfa42 Compare April 7, 2026 11:45
@Twixes Twixes changed the base branch from sig/component-inbox to graphite-base/1513 April 7, 2026 18:06
@Twixes Twixes force-pushed the graphite-base/1513 branch from d072d3d to bc8fae9 Compare April 7, 2026 18:06
@Twixes Twixes changed the base branch from graphite-base/1513 to main April 7, 2026 18:06
@Twixes Twixes force-pushed the sig/report-actions branch from 2fcfa42 to ad2f0db Compare April 7, 2026 19:57
@Twixes Twixes changed the base branch from main to graphite-base/1513 April 7, 2026 20:00
@Twixes Twixes force-pushed the sig/report-actions branch from ad2f0db to e7e2514 Compare April 7, 2026 20:00
@Twixes Twixes changed the base branch from graphite-base/1513 to sig/component-inbox April 7, 2026 20:00
@Twixes Twixes changed the title sig: add modifying report state via inbox feat(sig): add modifying report state via inbox Apr 7, 2026
@oliverb123 oliverb123 changed the base branch from sig/component-inbox to graphite-base/1513 April 7, 2026 21:49
PascalCase file names are reserved for files exporting React components.
This file only exports constants, so use kebab-case instead.

Co-authored-by: Michael Matloka <dev@twixes.com>
@oliverb123 oliverb123 force-pushed the sig/report-actions branch from e7e2514 to efb0df6 Compare April 7, 2026 21:50
@oliverb123 oliverb123 force-pushed the graphite-base/1513 branch from fbc1d50 to 1735579 Compare April 7, 2026 21:50
@graphite-app graphite-app bot changed the base branch from graphite-base/1513 to main April 7, 2026 21:50
@oliverb123 oliverb123 force-pushed the sig/report-actions branch from efb0df6 to f06467a Compare April 7, 2026 21:51
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