Skip to content

[PM-32149] Send email verification error dialogs#6535

Merged
aj-rosado merged 18 commits intomainfrom
PM-32149/send-email-verification-error-dialogs
Feb 19, 2026
Merged

[PM-32149] Send email verification error dialogs#6535
aj-rosado merged 18 commits intomainfrom
PM-32149/send-email-verification-error-dialogs

Conversation

@aj-rosado
Copy link
Contributor

🎟️ Tracking

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

📔 Objective

Add validation dialogs on invalid or empty emails

📸 Screenshots

Empty email addresses error dialog Invalid email addresses error dialog

@aj-rosado aj-rosado requested review from a team and david-livefront as code owners February 13, 2026 19:40
@github-actions github-actions bot added app:password-manager Bitwarden Password Manager app context app:authenticator Bitwarden Authenticator app context labels Feb 13, 2026
@aj-rosado aj-rosado changed the base branch from main to PM-31614/update-add-edit-send-ui-email-auth February 13, 2026 19:41
@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

Logo
Checkmarx One – Scan Summary & Details531afb24-048d-4680-ac53-ab504deb904a

Great job! No new security vulnerabilities introduced in this pull request

@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

❌ Patch coverage is 93.75000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 86.38%. Comparing base (1a69362) to head (2f4aa24).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...tools/feature/send/addedit/AddEditSendViewModel.kt 93.75% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6535      +/-   ##
==========================================
+ Coverage   86.37%   86.38%   +0.01%     
==========================================
  Files         799      802       +3     
  Lines       56949    57081     +132     
  Branches     8253     8256       +3     
==========================================
+ Hits        49191    49311     +120     
- Misses       4892     4903      +11     
- Partials     2866     2867       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Base automatically changed from PM-31614/update-add-edit-send-ui-email-auth to main February 13, 2026 23:01
# Conflicts:
#	app/src/main/kotlin/com/x8bit/bitwarden/ui/tools/feature/send/addedit/AddEditSendViewModel.kt
#	app/src/main/kotlin/com/x8bit/bitwarden/ui/tools/feature/send/addedit/util/AddEditSendStateExtensions.kt
#	ui/src/main/res/values/strings.xml
@aj-rosado aj-rosado added this pull request to the merge queue Feb 19, 2026
Merged via the queue into main with commit 4ee5511 Feb 19, 2026
16 checks passed
@aj-rosado aj-rosado deleted the PM-32149/send-email-verification-error-dialogs branch February 19, 2026 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app:authenticator Bitwarden Authenticator app context app:password-manager Bitwarden Password Manager app context

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants