Skip to content

feat(inbox): capture dismissal_note in dismiss analytics event#2287

Merged
andrewm4894 merged 1 commit into
mainfrom
inbox-dismissal-note-analytics
May 21, 2026
Merged

feat(inbox): capture dismissal_note in dismiss analytics event#2287
andrewm4894 merged 1 commit into
mainfrom
inbox-dismissal-note-analytics

Conversation

@andrewm4894
Copy link
Copy Markdown
Member

Summary

  • Adds dismissal_note to the Inbox report action event when action_type = "dismiss" and the user typed a note in the dismiss dialog
  • Truncated to 1000 chars (postgres stores up to 4000) to keep event payloads bounded
  • Only attached when the trimmed note is non-empty

Why

Annika asked in Slack for a dismiss event "including their feedback" so she can read why users are dismissing inbox reports directly in PostHog instead of querying postgres_signals_signalreportartefact. We were already sending the note to the API (stored in content.note) but it wasn't on the analytics event.

Matches the precedent set by the unmerged Discuss button PR (which captures question_text truncated to 500 chars on the discuss action).

Test plan

  • Dismiss a report with a reason picked but no note → event has dismissal_reason and no dismissal_note key
  • Dismiss a report with a reason and a note → event has both, note value matches input, truncated at 1000 chars if longer
  • Dismiss a report with whitespace-only note → no dismissal_note attached
  • Snooze (i.e. reason resolves to snooze) → no dismissal-related props on the event

Annika wants to read the free-text feedback users leave when dismissing inbox
reports without having to query the postgres artefact table. The note was
already being sent to the API but dropped from the `Inbox report action`
event payload.

Truncated to 1000 chars to keep event payloads bounded (postgres stores up
to 4000). Only attached when the user actually typed something.
@andrewm4894 andrewm4894 self-assigned this May 21, 2026
@andrewm4894 andrewm4894 requested review from a team and annikaschmid May 21, 2026 14:06
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 21, 2026

Reviews (1): Last reviewed commit: "feat(inbox): capture dismissal_note in d..." | Re-trigger Greptile

@andrewm4894 andrewm4894 merged commit 5825867 into main May 21, 2026
15 checks passed
@andrewm4894 andrewm4894 deleted the inbox-dismissal-note-analytics branch May 21, 2026 14:41
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