Skip to content

[PM-21740 / PM-27878] Update join organization email templates#6941

Merged
r-tome merged 28 commits intomainfrom
ac/pm-21740/update-join-organization-email-templates
Feb 16, 2026
Merged

[PM-21740 / PM-27878] Update join organization email templates#6941
r-tome merged 28 commits intomainfrom
ac/pm-21740/update-join-organization-email-templates

Conversation

@r-tome
Copy link
Contributor

@r-tome r-tome commented Feb 3, 2026

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-28396
https://bitwarden.atlassian.net/browse/PM-27878

📔 Objective

Update organization invite email templates to use MJML-based designs.

📸 Screenshots

Screenshots for each organization plan type along with a zip file with html file + images

Enterprise/Teams - Existing User

enterprise-existing.zip

enterprise-existing html enterprise-existing(Pixel 7)
Enterprise/Teams - New User

enterprise-new.zip

enterprise-new html enterprise-new(Pixel 7)
Families - Existing User

family-existing.zip

family-existing html family-existing(Pixel 7)
Families - New User

family-new.zip
family-new html
family-new(Pixel 7)

Free - Existing User

free-existing.zip
free-existing html
free-existing(Pixel 7)

Free - New User

free-new.zip

free-new html free-new(Pixel 7)

⏰ Reminders before review

  • Contributor guidelines followed
  • All formatters and local linters executed and passed
  • Written new unit and / or integration tests where applicable
  • Protected functional changes with optionality (feature flags)
  • Used internationalization (i18n) for all UI strings
  • CI builds passed
  • Communicated to DevOps any deployment requirements
  • Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

  • 👍 (:+1:) or similar for great changes
  • 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
  • ❓ (:question:) for questions
  • 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
  • 🎨 (:art:) for suggestions / improvements
  • ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
  • 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
  • ⛏ (:pick:) for minor or nitpick changes

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

Logo
Checkmarx One – Scan Summary & Details0439d15b-92f8-4e6f-88ca-0e7fdc4d8b97

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

…ogic by removing the OrganizationCategory enum and consolidating plan type checks. Introduce a new method for formatting expiration dates to enhance code clarity and maintainability.
…es for organization invites, allowing support for TeamsStarter, TeamsStarter2023, and Custom plans in the invitation logic.
…g logic for various plan types, including Enterprise, Teams, Families, Free, and Custom plans, based on user existence and feature flags.
@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

❌ Patch coverage is 96.80365% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.46%. Comparing base (ca35b9e) to head (8c5c254).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...sers/InviteUsers/SendOrganizationInvitesCommand.cs 97.35% 3 Missing and 2 partials ⚠️
...le/Services/Implementations/OrganizationService.cs 60.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6941      +/-   ##
==========================================
+ Coverage   56.35%   56.46%   +0.11%     
==========================================
  Files        1990     1996       +6     
  Lines       87832    88039     +207     
  Branches     7835     7845      +10     
==========================================
+ Hits        49494    49715     +221     
+ Misses      36503    36487      -16     
- Partials     1835     1837       +2     

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

r-tome and others added 9 commits February 5, 2026 13:50
…out consistency across various user types, including adjustments to padding, font weights, and vertical alignment for a more polished appearance.
…stency and layout across various user types, including adjustments to padding, font families, and visibility of icon rows for a more polished appearance.
…emplates

# Conflicts:
#	src/Core/Constants.cs
<mj-column width="85%" vertical-align="middle" css-class="mj-bw-ac-icon-row-text-column">
${headAnchorElement}
<mj-text css-class="mj-bw-ac-icon-row-text" padding="0px 0px 0px 0px" ${BODY_TEXT_STYLES}>
<span class="mj-bw-ac-icon-row-bullet" style="display: none;">&#8226;&nbsp;</span>${this.getAttribute("text")}
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like mj-bw-ac-icon-row-bullet and style="display: none;" have the same setting. Doesn’t that mean they’ll override each other?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

display: none; is the default for Desktop, mj-bw-ac-icon-row-bullet overrides it for mobile

…bility of bullet points by changing display properties and adding inline text spans.
@r-tome r-tome marked this pull request as ready for review February 13, 2026 15:44
@r-tome r-tome requested review from a team as code owners February 13, 2026 15:44
@r-tome r-tome requested review from eliykat and kdenney February 13, 2026 15:44
@r-tome r-tome changed the title [PM-21740] Update join organization email templates [PM-21740 / PM-27878] Update join organization email templates Feb 13, 2026
@r-tome r-tome requested a review from JimmyVo16 February 13, 2026 15:47
JimmyVo16
JimmyVo16 previously approved these changes Feb 13, 2026
kdenney
kdenney previously approved these changes Feb 13, 2026
Copy link
Contributor

@kdenney kdenney left a comment

Choose a reason for hiding this comment

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

Billing change looks good.

…emplates

# Conflicts:
#	src/Core/MailTemplates/Mjml/emails/AdminConsole/components/mj-bw-ac-icon-row.js
@r-tome r-tome dismissed stale reviews from kdenney and JimmyVo16 via a7f65db February 13, 2026 18:27
@r-tome r-tome requested review from JimmyVo16 and kdenney February 13, 2026 18:27
@sonarqubecloud
Copy link

@r-tome r-tome merged commit e65ed48 into main Feb 16, 2026
40 of 41 checks passed
@r-tome r-tome deleted the ac/pm-21740/update-join-organization-email-templates branch February 16, 2026 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants