Skip to content

ref(flags): Remove organizations:autofix-browser-notifications registration - #129456

Open
cursor[bot] wants to merge 1 commit into
masterfrom
claude-rm-autofix-browser-notifications-registration
Open

cursor[bot] wants to merge 1 commit into
masterfrom
claude-rm-autofix-browser-notifications-registration

Conversation

@cursor

@cursor cursor Bot commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

Remove the temporary.py registration for organizations:autofix-browser-notifications. The flag was rolled out to GA in the automator in getsentry/sentry-options-automator#9026 on 8/3, and hasn't been touched since.

Last of three, merge in order:

  1. ref(flags): Graduate organizations:autofix-browser-notifications in the UI #129455 — frontend reads
  2. getsentry/sentry-options-automatorfix(integrations) Add pull request links for gitlab #10417 — flagpole value
  3. this PR

The flag has no backend readers, so this does not depend on #129455 for green CI; the ordering is so the flagpole block is never orphaned and api_expose keeps emitting the flag while the UI still reads it.

The Disable for sandbox org segment the flagpole block carried is preserved in code as !isDemoModeActive() by #129455.

Open in Web View Automation 

…ration

GA in the automator since #9026 (2026-08-03). Reads removed in #129455,
flagpole value in getsentry/sentry-options-automator#10417.

Co-authored-by: charlie.luo <charlie.luo@sentry.io>
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 1, 2026
@cursor
cursor Bot marked this pull request as ready for review November 1, 2026 20:39
@cursor
cursor Bot requested a review from ryan953 November 1, 2026 20:39

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@getsentry/machine-learning-ai — last of three for organizations:autofix-browser-notifications, opened by the stale-flag-cleanup automation. @ryan953 is requested as reviewer since he created and rolled out the flag.

Merge this one last: #129455 (frontend reads) → getsentry/sentry-options-automator#10417 (flagpole value) → this PR. The flag has no backend readers, so green CI here does not depend on #129455 — the ordering exists so the flagpole block is never orphaned and so api_expose keeps emitting the flag while the UI still reads it.

The flagpole block carried a Disable for sandbox org segment above its GA segment; #129455 preserves that as !isDemoModeActive() and explains why. Worth reading that PR's discussion before merging this one.

CI is green apart from getsentry dispatch, which gates on the author having write permission and always fails for cursor[bot] — a collaborator needs to add the Trigger: getsentry tests label.

View PR

Open in Web View Automation 

Sent by Cursor Automation: Feature flag cleanup

This branch has not been deployed

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

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant