Skip to content

feat(mobile): add Discuss button to inbox reports#2409

Merged
Gilbert09 merged 1 commit into
mainfrom
posthog-code/mobile-inbox-discuss-button
May 28, 2026
Merged

feat(mobile): add Discuss button to inbox reports#2409
Gilbert09 merged 1 commit into
mainfrom
posthog-code/mobile-inbox-discuss-button

Conversation

@Gilbert09
Copy link
Copy Markdown
Member

Summary

  • Ports the desktop "Discuss" affordance (feat(code): add Discuss button to inbox reports #2241) to mobile: tapping Discuss on an inbox report opens a sheet where the user can optionally type a first question, then routes to the new-task composer with the agent-fetch-the-report prompt prefilled and the signal_report id attached.
  • Moves buildDiscussReportPrompt into @posthog/shared and parameterises it on the deep-link URL so each host passes its own scheme (posthog-code:// on desktop, posthog:// on mobile) without forking the wording. The desktop wrapper keeps its previous isDevBuild signature so all existing callers stay unchanged.
  • Fires the same Inbox report action analytics event desktop does with action_type: "discuss", surface: "detail_pane", has_question, and the report id, so the discuss surface shows up alongside the other inbox actions.

Test plan

  • Open an inbox report on mobile, tap Discuss: sheet appears with text input
  • Submit empty → composer opens with the "brief readout" variant of the prompt and the repo + signalReport params prefilled
  • Submit with a question → composer opens with the "answer this first" variant
  • Confirm the new task is tagged with origin_product: "signal_report" and signal_report_id on the API side once sent
  • On desktop, confirm the existing Discuss button still works (the wrapper now delegates to the shared util)
  • pnpm --filter code test buildDiscussReportPrompt still passes

Mirrors the desktop PR (#2241) on mobile. Tapping Discuss on
an inbox report opens a sheet where the user can optionally type a first
question, then routes them to the new-task composer with the
agent-fetch-the-report prompt prefilled and the signal_report id attached.

The prompt builder moves to @posthog/shared and is parameterised on the
deep-link URL so each host can pass its own scheme (posthog-code:// on
desktop, posthog:// on mobile) without forking the wording.

Fires the same "Inbox report action" event desktop does with
action_type: "discuss" and the report id, so the discuss surface shows
up alongside the other inbox actions in analytics.

Generated-By: PostHog Code
Task-Id: bad407bc-3425-4688-87ca-3b2d61673b5c
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 28, 2026

Reviews (1): Last reviewed commit: "feat(mobile): add Discuss button to inbo..." | Re-trigger Greptile

@Gilbert09 Gilbert09 added the Stamphog This will request an autostamp by stamphog on small changes label May 28, 2026
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean additive feature: utility logic extracted to shared package with identical behavior, mobile UI follows the existing DismissReportSheet pattern, null guard on report is in place before the new sheet is rendered.

@Gilbert09 Gilbert09 merged commit 0760503 into main May 28, 2026
17 checks passed
@Gilbert09 Gilbert09 deleted the posthog-code/mobile-inbox-discuss-button branch May 28, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant