Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add strong to the list of tags that can have style #35864

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MartinRiese
Copy link
Contributor

@MartinRiese MartinRiese commented Feb 27, 2025

Product Description

When added additional styles like text color to bold/strong text in rich text emails the styles was stripped out. Add style and class to the allowed attributes for strong like we have from em and other tags

Technical Summary

Follow up for https://dimagi.atlassian.net/browse/USH-5567

Feature Flag

rich_text_emails

Safety Assurance

Safety story

Automated test coverage

QA Plan

https://dimagi.atlassian.net/browse/QA-7532

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

Labels & Review

  • Risk label is set correctly
  • The set of people pinged as reviewers is appropriate for the level of risk of the change

@MartinRiese MartinRiese marked this pull request as ready for review March 3, 2025 20:52
@MartinRiese MartinRiese requested review from a team, minhaminha and Robert-Costello and removed request for a team March 3, 2025 20:52
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.

2 participants