Skip to content

Conversation

@LetItRock
Copy link
Contributor

What changed? Why was the change needed?

Screenshots

Expand for optional sections

Related enterprise PR

Special notes for your reviewer

@netlify
Copy link

netlify bot commented Jan 21, 2026

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

Name Link
🔨 Latest commit 56df9f5
🔍 Latest deploy log https://app.netlify.com/projects/dashboard-v2-novu-staging/deploys/69711351286f470008fe6a92

@github-actions
Copy link
Contributor

Hey there and thank you for opening this pull request! 👋

We require pull request titles to follow specific formatting rules and it looks like your proposed title needs to be adjusted.

Your PR title is: fix(worker): disable resolving external css from the link tags

Requirements:

  1. Follow the Conventional Commits specification
  2. As a team member, include Linear ticket ID at the end: fixes TICKET-ID or include it in your branch name

Expected format: feat(scope): Add fancy new feature fixes NOV-123

Details:

PR title must end with 'fixes TICKET-ID' (e.g., 'fixes NOV-123') or include ticket ID in branch name

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 21, 2026

Walkthrough

A configuration option has been added to the CSS inlining process in the email message sending workflow. The applyLinkTags: false option is now passed to the inlineCss function, which prevents the inlining process from applying link tags. This represents a behavioral change to how CSS inlining is handled during email message composition, with no modifications to the surrounding logic.

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description contains only template placeholders with no actual details about the change, making it impossible to assess relevance to the changeset. Provide a meaningful description explaining why this change was needed and what problem it solves.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: disabling external CSS resolution from link tags during email CSS inlining.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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 369bd5a into next Jan 21, 2026
29 checks passed
@LetItRock LetItRock deleted the fix-inline-css-disable-resolving-external-css branch January 21, 2026 18:08
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.

2 participants