Skip to content

feat: Remove organizations:incidents flag - #4487

Open
RudraPatel2003 wants to merge 1 commit into
masterfrom
rudra-remove-organization-incidents
Open

feat: Remove organizations:incidents flag#4487
RudraPatel2003 wants to merge 1 commit into
masterfrom
rudra-remove-organization-incidents

Conversation

@RudraPatel2003

@RudraPatel2003 RudraPatel2003 commented Aug 24, 2026

Copy link
Copy Markdown
Member

Cleanup that will be done after these 3 steps

Fixes ISWF-3270

image

@cursor cursor 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit ff9db54. Configure here.

"organizations:discover",
"organizations:global-views",
"organizations:issue-views",
"organizations:incidents",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Metric alerts flag wrongly removed

High Severity

Removing organizations:incidents from the example SENTRY_FEATURES list disables metric alerts for new self-hosted installs. That flag still exists in Sentry with default False and still gates incident and metric-alert APIs, so copies of this example leave the feature off.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit ff9db54. Configure here.

@aldy505

aldy505 commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

@RudraPatel2003 Hi, can you provide any justification or why you're removing this flag?

RudraPatel2003 commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

The flag gates metric monitors, which are now available even to free-tier users. The dIscussion on removing this flag was started in the following PR

The flag checks have all been removed from the sentry codebase already. I am in the process of now removing the flag itself

getsentry/getsentry#20511

@linear-code

linear-code Bot commented Aug 25, 2026

Copy link
Copy Markdown

ISWF-3270

@saponifi3d saponifi3d 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.

the bot is generally correct -- but once we merge the open PRs this should be fine.

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants