chore(dashboards): remove text widget flag defintion#116212
Conversation
github-actions
Bot
added
the
Scope: Backend
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
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 d7a5c76. Configure here.
| manager.add("organizations:dashboards-prebuilt-insights-dashboards", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=True) | ||
| # Enable the details widget for dashboards | ||
| manager.add("organizations:dashboards-details-widget", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=True) | ||
| # Enable text widgets for dashboards |
There was a problem hiding this comment.
Removing flag definition breaks text widget validation
High Severity
Removing the organizations:dashboards-text-widgets flag definition from temporary.py while the backend still gates on it in _validate_text_widget will break all text widget creation and editing. When features.has() is called for an unregistered flag, _get_feature_class raises FeatureNotRegistered, which is caught by the exception handler in has() and returns False. This causes the serializer to always reject text widgets with "Text widgets are not enabled". The frontend typeSelector.tsx also checks this flag and will hide the text widget option.
Reviewed by Cursor Bugbot for commit d7a5c76. Configure here.
There was a problem hiding this comment.
other prs remove it from the code
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
2 participants
should be merged AFTER https://github.com/getsentry/sentry-options-automator/pull/7935