Skip to content

Conversation

@billyvg
Copy link
Member

@billyvg billyvg commented Jan 5, 2026

The new design changes werent that major so we are removing this flag. (usage was removed in #104246, flag removed in https://github.com/getsentry/sentry-options-automator/pull/6052)

The new design changes werent that major so we are removing this flag. (usage was removed in #104246)
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 5, 2026
@billyvg billyvg marked this pull request as ready for review January 6, 2026 16:18
# Enable core Session Replay link in the sidebar
manager.add("organizations:session-replay-ui", OrganizationFeature, FeatureHandlerStrategy.INTERNAL, default=True, api_expose=True)
# Enable new UI for replay details page
manager.add("organizations:replay-details-new-ui", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=True)
Copy link
Contributor

Choose a reason for hiding this comment

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

Removing flag breaks new UI - always shows old UI

The feature flag replay-details-new-ui is being removed from the backend, but frontend code in static/app/views/replays/details.tsx still checks organization.features.includes('replay-details-new-ui') to decide which UI to show. After this removal, the check will always return false, causing users to always see the old UI instead of the new UI. The frontend usage needs to be removed first or simultaneously, making the new UI the unconditional default.

Fix in Cursor Fix in Web

@billyvg billyvg requested a review from a team January 6, 2026 18:14
@billyvg billyvg merged commit 0dfadd3 into master Jan 7, 2026
67 checks passed
@billyvg billyvg deleted the billy/replay-remove-details-new-ui-flag branch January 7, 2026 14:45
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.

3 participants