Skip to content

feat(inbox): suppress dialog with dismissal feedback#2053

Draft
Twixes wants to merge 1 commit intomainfrom
posthog-code/suppress-dialog-with-dismissal-feedback
Draft

feat(inbox): suppress dialog with dismissal feedback#2053
Twixes wants to merge 1 commit intomainfrom
posthog-code/suppress-dialog-with-dismissal-feedback

Conversation

@Twixes
Copy link
Copy Markdown
Member

@Twixes Twixes commented May 6, 2026

Summary

  • Replaces the bulk-suppress confirmation in the inbox toolbar — and adds a new Suppress button to the top of the report detail pane — with a dialog that captures why a report is being suppressed.
  • The dialog has six radio options ("Already fixed elsewhere", "Agent's analysis is wrong", "Won't fix - intentional behavior", "Won't fix - issue is real but irrelevant", "I'm not the right reviewer", "Other") plus an optional free-form add detail textarea.
  • Suppression continues to flow through the existing POST /api/projects/:team_id/signals/reports/:id/state/ endpoint, with two new optional body fields (dismissal_reason, dismissal_note). The backend persists those as a new dismissal-type SignalReportArtefact so the rationale survives status changes and stacks across multiple dismissals. The companion backend PR is in PostHog/posthog.

Notable changes

  • New SuppressDialog component (apps/code/src/renderer/features/inbox/components/SuppressDialog.tsx) — uses Radix Dialog + RadioGroup + TextArea.
  • useInboxBulkActions.suppressSelected(dismissal?) now takes an optional { reason, note } payload.
  • posthogClient.updateSignalReportState accepts and forwards dismissal_reason / dismissal_note.
  • DismissalArtefact shared type + parser branch in the artefact normalizer.
  • Suppress button on the report detail header closes the pane on success.

Test plan

  • Bulk-select reports in the inbox, click Suppress, pick a reason, optionally add detail, confirm — reports become suppressed and the dismissal artefact appears in the artefacts list.
  • Open a single report in the detail pane, click Suppress in the header, confirm with no note — succeeds and pane closes.
  • Try to confirm without picking a reason — button is disabled with a "you haven't picked a reason" tooltip.
  • Verify the existing Snooze / Delete flows are unaffected (Delete still uses the original `AlertDialog`).
  • Confirm typecheck and biome pass.

Created with PostHog Code

Add a Suppress button to the top of the report detail pane and replace
the bulk-suppress confirmation in the toolbar with a dialog that asks
why the user is suppressing. The dialog offers six radio reasons plus an
optional free-form note. Selections are sent alongside the state
transition and persisted server-side as a `dismissal`-type report
artefact so the rationale survives status changes and can stack over
time.

Generated-By: PostHog Code
Task-Id: d3d675ab-707a-4a35-939b-9a9ab0f5c5b5
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.

1 participant