Skip to content

ref(performance): Remove Transaction Summary's "Tags" tab#119080

Open
mjq wants to merge 1 commit into
masterfrom
mjq/txn-summary-tags-tab-removal
Open

ref(performance): Remove Transaction Summary's "Tags" tab#119080
mjq wants to merge 1 commit into
masterfrom
mjq/txn-summary-tags-tab-removal

Conversation

@mjq

@mjq mjq commented Jul 6, 2026

Copy link
Copy Markdown
Member

This tab has been unconditionally hidden for months now and depends on the deprecated transactions dataset. Remove the code.


🤖: Claude made the code changes, guided by me + knip.

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 6, 2026
@mjq mjq marked this pull request as ready for review July 6, 2026 20:40
@mjq mjq requested review from a team as code owners July 6, 2026 20:40

@adrianviquez adrianviquez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lgtm, preview deployment doesn't look broken to me either. Found this test https://github.com/getsentry/sentry/blob/master/static/app/views/performance/transactionSummary/header.spec.tsx#L87 that I believe is hiding this tab, but would be dead code after this goes in, so would delete that. Otherwise 🚢

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants