Skip to content

[PM-28781] Remove Send UI refresh feature flag#20753

Open
mcamirault wants to merge 1 commit into
mainfrom
tools/pm-28178/remove-send-ui-refresh-ff
Open

[PM-28781] Remove Send UI refresh feature flag#20753
mcamirault wants to merge 1 commit into
mainfrom
tools/pm-28178/remove-send-ui-refresh-ff

Conversation

@mcamirault
Copy link
Copy Markdown
Contributor

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-28178

📔 Objective

This PR removes all display logic behind the pm-28175-send-ui-refresh feature flag from our clients

@mcamirault mcamirault requested a review from a team as a code owner May 20, 2026 17:15
@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

❌ Patch coverage is 30.00000% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 47.46%. Comparing base (0345984) to head (fe314fc).
⚠️ Report is 29 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
apps/web/src/app/tools/send/send.component.ts 0.00% 4 Missing ⚠️
...tools/send/new-send/new-send-dropdown.component.ts 50.00% 1 Missing and 2 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main   #20753   +/-   ##
=======================================
  Coverage   47.46%   47.46%           
=======================================
  Files        4003     4003           
  Lines      123145   123128   -17     
  Branches    18945    18940    -5     
=======================================
- Hits        58447    58441    -6     
+ Misses      60272    60261   -11     
  Partials     4426     4426           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@itsadrago itsadrago changed the title [PM-28178] Remove Send UI refresh feature flag [PM-28781] Remove Send UI refresh feature flag May 21, 2026
@mcamirault mcamirault added the needs-qa Marks a PR as requiring QA approval label May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-qa Marks a PR as requiring QA approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants