Repository navigation
ref(flags): Remove organizations:tracemetrics-pii-scrubbing-ui gate (frontend) - #127147
cursor[bot] wants to merge 1 commit into
Conversation
… (frontend) The flag has been fully rolled out (segment with no conditions at 100% rollout) in sentry-options-automator since #7574 (2026-04-28), so collapse the predicate to the surviving `tracemetrics-enabled` gate. Co-authored-by: charlie.luo <charlie.luo@sentry.io>
There was a problem hiding this comment.
@getsentry/data-browsing @getsentry/explore — review please. data-browsing is the owner.team on the flagpole entry; @getsentry/explore owns /static/app/views/explore/ in CODEOWNERS.
Merge order — this is PR 1 of 3:
- this PR (frontend gate)
- getsentry/sentry-options-automator#10134 (flagpole value)
- #127148 (
temporary.pyregistration) — merge last, after #10134 has deployed everywhere
CI is green apart from getsentry dispatch, which fails on every PR this bot opens because the check gates on the author having write permission. A collaborator needs to add the Trigger: getsentry tests label if you want those to run.
Note for reviewers: canUseMetricsPiiScrubbingUI is kept rather than deleted because it still has two callers in static/app/views/settings/components/dataScrubbing/. That matches #115018, which collapsed canUseMetricsAlertsUI in this same file. Happy to inline it at the call sites instead if you'd prefer.
Sent by Cursor Automation: Feature flag cleanup
Remove
organizations:tracemetrics-pii-scrubbing-ui. This flag was fully rolled out in the automator in PR #7574 on 4/28/26 (segment with no conditions at 100% rollout), and hasn't been touched since.The only reader was
canUseMetricsPiiScrubbingUIinstatic/app/views/explore/metrics/metricsFlags.tsx, added in #113078. The predicate still has two callers instatic/app/views/settings/components/dataScrubbing/, so its body is collapsed to the survivingtracemetrics-enabledgate rather than deleted — same shape as #115018, which graduatedcanUseMetricsAlertsUIin this file.First of three PRs (reads -> automator value -> registration):
temporary.pyregistrationVerified locally:
pnpm run typecheckclean, 353 tests pass acrossdataScrubbing/andexplore/metrics/, oxlint + oxfmt clean.Legal Boilerplate
Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.