Skip to content

fix(ui): remove unfinished comments tab - #3948

Merged
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/remove-comments-tab
Jul 29, 2026
Merged

fix(ui): remove unfinished comments tab#3948
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/remove-comments-tab

Conversation

@puemos

@puemos puemos commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Problem

The Comments tab is visible before it is ready for use.

Why: Keep unfinished functionality out of the activity panel until it is ready to ship.

Changes

Removed the Comments tab and its dedicated rendering path from the activity panel.

How did you test this?

  • pnpm exec biome check packages/ui/src/features/canvas/components/ActivityPanel.tsx
  • git diff --check
  • UI package typecheck attempted, but workspace dependency build outputs were unavailable in the fresh clone.

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Created with PostHog Code

Generated-By: PostHog Code
Task-Id: df260293-b59c-4a18-9654-d1748366ce52
@trunk-io

trunk-io Bot commented Jul 29, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 63d5736.

@puemos puemos self-assigned this Jul 29, 2026
@puemos
puemos marked this pull request as ready for review July 29, 2026 13:35
@puemos puemos added the Stamphog This will request an autostamp by stamphog on small changes label Jul 29, 2026
@puemos puemos changed the title Remove unfinished comments tab fix(ui): remove unfinished comments tab Jul 29, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Purely additive removal of an unfinished UI tab in the activity panel, matches description exactly, no risky territory touched, and author has strong familiarity with the file.

  • Author wrote 100% of the modified lines and has 13 merged PRs in these paths (familiarity STRONG).
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 29L, 1F substantive — within ceiling
tier T1-agent / T1b-small (29L, 1F, single-area, unknown)
stamphog 2.0.0b3 .stamphog/policy.yml @ 7408a79 · reviewed head 63d5736

@trunk-io
trunk-io Bot merged commit 446246f into main Jul 29, 2026
41 checks passed
@trunk-io
trunk-io Bot deleted the posthog-code/remove-comments-tab branch July 29, 2026 14:02
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