Skip to content

ref(slack): remove widget unfurl feature flags#116128

Merged
DominikB2014 merged 2 commits into
masterfrom
dbuszowiecki/remove-widget-unfurl-flags
May 25, 2026
Merged

ref(slack): remove widget unfurl feature flags#116128
DominikB2014 merged 2 commits into
masterfrom
dbuszowiecki/remove-widget-unfurl-flags

Conversation

@DominikB2014
Copy link
Copy Markdown
Contributor

@DominikB2014 DominikB2014 commented May 25, 2026

Summary

  • Removes the organizations:data-browsing-widget-unfurl and organizations:dashboards-widget-unfurl registrations from src/sentry/features/temporary.py.
  • Switches the Slack unfurl flag gates over to the base feature flags for each product (organizations:visibility-explore-view for explore, organizations:dashboards-basic for dashboards) in explore.py, dashboards.py, and webhooks/event.py. This keeps the existing "only unfurl when the org has the underlying product" guard while removing the unfurl-specific flags.

Removes the `organizations:data-browsing-widget-unfurl` and
`organizations:dashboards-widget-unfurl` flag gates from the Slack
unfurl handlers (now always-on) and drops the registrations from
`temporary.py`. The identity-prompt path for unlinked users on
explore/dashboards links is preserved.

DAIN-1689
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 25, 2026
@DominikB2014 DominikB2014 marked this pull request as ready for review May 25, 2026 17:04
@DominikB2014 DominikB2014 requested a review from a team as a code owner May 25, 2026 17:04
@DominikB2014 DominikB2014 merged commit 2e4a45d into master May 25, 2026
85 checks passed
@DominikB2014 DominikB2014 deleted the dbuszowiecki/remove-widget-unfurl-flags branch May 25, 2026 17:57
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.

2 participants