Skip to content

Conversation

@JonasBa
Copy link
Member

@JonasBa JonasBa commented Dec 11, 2025

UI2 has been rolled out and the feature flags are no longer necessary

ref DE-657, Fix DE-658

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 11, 2025
@linear
Copy link

linear bot commented Dec 11, 2025

Comment on lines 215 to 220
"prefersIssueDetailsStreamlinedUI": options.get(
"prefers_issue_details_streamlined_ui"
),
"prefersChonkUI": options.get("prefers_chonk_ui"),
}

d["flags"] = {"newsletter_consent_prompt": bool(obj.flags.newsletter_consent_prompt)}

This comment was marked as outdated.

@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #104828      +/-   ##
===========================================
+ Coverage   80.45%    80.53%   +0.08%     
===========================================
  Files        9396      9377      -19     
  Lines      403551    402474    -1077     
  Branches    25904     25694     -210     
===========================================
- Hits       324668    324138     -530     
+ Misses      78448     77901     -547     
  Partials      435       435              

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 12, 2025
@github-actions
Copy link
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

"clock24Hours": True,
"extra": True,
"prefersIssueDetailsStreamlinedUI": True,
"prefersChonkUI": True,
Copy link
Member Author

Choose a reason for hiding this comment

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

This is the only frontend change which will be removed once the dependent PR is merged

@JonasBa
Copy link
Member Author

JonasBa commented Dec 12, 2025

Typescript checks will pass once the dependent PR lands

@JonasBa JonasBa removed the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 15, 2025
@JonasBa JonasBa merged commit 35a40c6 into master Dec 15, 2025
69 checks passed
@JonasBa JonasBa deleted the jb/chonk/flags branch December 15, 2025 21:15
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