feat: Remove organizations:incidents flag - #4487
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ 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", |
There was a problem hiding this comment.
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.
Reviewed by Cursor Bugbot for commit ff9db54. Configure here.
|
@RudraPatel2003 Hi, can you provide any justification or why you're removing this flag? |
|
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 |
saponifi3d
left a comment
There was a problem hiding this comment.
the bot is generally correct -- but once we merge the open PRs this should be fine.


Cleanup that will be done after these 3 steps
Fixes ISWF-3270