Skip to content

PM-29781 removed feature flag and corresponding code#20487

Open
bmbitwarden wants to merge 2 commits intomainfrom
PM-29781-Remove-the-feature-flag-for-email-verification-clients-second
Open

PM-29781 removed feature flag and corresponding code#20487
bmbitwarden wants to merge 2 commits intomainfrom
PM-29781-Remove-the-feature-flag-for-email-verification-clients-second

Conversation

@bmbitwarden
Copy link
Copy Markdown
Contributor

🎟️ Tracking

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

📔 Objective

Remove the feature flag pm-19051-send-email-verification from clients and its corresponding stale code

📸 Screenshots

Screen.Recording.2026-04-30.at.5.44.26.PM.mov
Screen.Recording.2026-04-30.at.5.42.23.PM.mov

@bmbitwarden bmbitwarden requested a review from a team as a code owner May 4, 2026 11:23
@codecov
Copy link
Copy Markdown

codecov Bot commented May 4, 2026

Codecov Report

❌ Patch coverage is 28.57143% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 47.12%. Comparing base (90ca772) to head (c597b90).
⚠️ Report is 12 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
.../app/tools/send/send-access/send-view.component.ts 0.00% 5 Missing ⚠️
...ols/send/send-access/send-access-file.component.ts 0.00% 2 Missing ⚠️
apps/cli/src/tools/send/send.program.ts 0.00% 1 Missing ⚠️
.../app/tools/send/send-access/send-auth.component.ts 0.00% 1 Missing ⚠️
.../components/send-details/send-details.component.ts 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #20487      +/-   ##
==========================================
+ Coverage   47.10%   47.12%   +0.01%     
==========================================
  Files        3948     3948              
  Lines      119684   119559     -125     
  Branches    18344    18309      -35     
==========================================
- Hits        56376    56341      -35     
+ Misses      59074    58994      -80     
+ Partials     4234     4224      -10     

☔ 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.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 4, 2026

@bmbitwarden bmbitwarden added the needs-qa Marks a PR as requiring QA approval label May 5, 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