Skip to content

fix(api-service): decode html entities in subject and maily html fixes NV-7039#9842

Merged
LetItRock merged 1 commit intonextfrom
nv-7039-decode-html-entities-for-subject-and-maily-html
Jan 19, 2026
Merged

fix(api-service): decode html entities in subject and maily html fixes NV-7039#9842
LetItRock merged 1 commit intonextfrom
nv-7039-decode-html-entities-for-subject-and-maily-html

Conversation

@LetItRock
Copy link
Contributor

What changed? Why was the change needed?

API: decode HTML entities in email subject and rendered Maily HTML

Screenshots

Screenshot 2026-01-16 at 13 43 49 Screenshot 2026-01-16 at 13 40 55 Screenshot 2026-01-16 at 13 40 19

@LetItRock LetItRock self-assigned this Jan 16, 2026
@linear
Copy link

linear bot commented Jan 16, 2026

@netlify
Copy link

netlify bot commented Jan 16, 2026

Deploy Preview for dashboard-v2-novu-staging canceled.

Name Link
🔨 Latest commit ae1df7f
🔍 Latest deploy log https://app.netlify.com/projects/dashboard-v2-novu-staging/deploys/696a35fa9e1aa7000854bfc7

@github-actions github-actions bot changed the title fix(api-service): decode html entities in subject and maily html fix(api-service): decode html entities in subject and maily html fixes NV-7039 Jan 16, 2026
@github-actions
Copy link
Contributor

PR title automatically updated!

I found the Linear ticket ID fix(api-service): decode html entities in subject and maily html fixes NV-7039 in your branch name and automatically added it to your PR title.

Updated title: fix(api-service): decode html entities in subject and maily html fixes NV-7039

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 16, 2026

Walkthrough

This pull request adds HTML entity decoding to the email output rendering process. The decodeHTML function from the entities module is integrated into the email renderer's subject and body processing methods to remove HTML escape sequences. Dependencies are reorganized with the addition of the entities module. End-to-end tests are added to verify that HTML entities in email subjects and bodies are correctly decoded and rendered across HTML editor and block editor workflows.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description is related to the changeset, clearly stating the change needed (decode HTML entities in email subject and rendered Maily HTML) and includes supporting screenshots.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title directly describes the main change: decoding HTML entities in email subjects and Maily HTML output, which is supported by all three modified files.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@LetItRock LetItRock merged commit 6ca2b34 into next Jan 19, 2026
29 checks passed
@LetItRock LetItRock deleted the nv-7039-decode-html-entities-for-subject-and-maily-html branch January 19, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant