Skip to content

[PM-41628] Update email copy - #8167

Merged
BTreston merged 1 commit into
mainfrom
ac/pm-41628-update-email-copy
Aug 7, 2026
Merged

[PM-41628] Update email copy#8167
BTreston merged 1 commit into
mainfrom
ac/pm-41628-update-email-copy

Conversation

@BTreston

@BTreston BTreston commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

🎟️ Tracking

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

📔 Objective

updates email copy

📸 Screenshots

@BTreston
BTreston requested review from a team and sven-bitwarden August 7, 2026 13:59
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR updates the body copy of the provider user confirmation email in both the HTML and plain-text Handlebars templates, changing "This email is to notify you that you have been confirmed as a user of {{ProviderName}}" to "You've been confirmed as a user of {{ProviderName}}". The change also incidentally strips the UTF-8 BOM from both templates and adds a trailing newline to the text template, which brings them in line with the repository .editorconfig (charset = utf-8, insert_final_newline = true for non-.cs files). No template variables, escaping, or rendering logic were altered — {{ProviderName}} remains HTML-escaped by Handlebars and sanitized via CoreHelpers.SanitizeForEmail in HandlebarsMailService.SendProviderConfirmedEmailAsync. No tests or other code reference the previous copy string.

Code Review Details

No findings.

@BTreston BTreston added the t:bugfix Change Type - Bugfix label Aug 7, 2026
@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.04%. Comparing base (73ed2f6) to head (48e29db).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #8167       +/-   ##
===========================================
+ Coverage   15.10%   63.04%   +47.93%     
===========================================
  Files        1417     2315      +898     
  Lines       61422   100510    +39088     
  Branches     4901     9043     +4142     
===========================================
+ Hits         9279    63364    +54085     
+ Misses      51978    34957    -17021     
- Partials      165     2189     +2024     

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@BTreston
BTreston merged commit 4f8c0f0 into main Aug 7, 2026
47 of 48 checks passed
@BTreston
BTreston deleted the ac/pm-41628-update-email-copy branch August 7, 2026 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:bugfix Change Type - Bugfix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants