Skip to content

feat: MJML docs#1355

Merged
cjbell merged 3 commits intomainfrom
cb-mjml
Mar 18, 2026
Merged

feat: MJML docs#1355
cjbell merged 3 commits intomainfrom
cb-mjml

Conversation

@cjbell
Copy link
Contributor

@cjbell cjbell commented Mar 18, 2026

Description

MJML docs

@vercel
Copy link

vercel bot commented Mar 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 18, 2026 8:42pm

Request Review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Risk HIGH: Adds MJML documentation with a new content page, cross-references across multiple docs sections, and registers XML/MJML syntax highlighting in the shared CodeBlock.tsx component.

Reasons

  • components/ui/CodeBlock.tsx is modified — a .tsx file in components/ triggers HIGH risk per classification rules
  • CodeBlock.tsx is a shared, reusable component used across all code blocks site-wide (explicitly listed as a HIGH trigger)
  • New MDX page added at content/integrations/email/mjml.mdx (78 lines, MEDIUM trigger)
  • Sidebar navigation modified in data/sidebars/integrationsSidebar.ts (MEDIUM trigger)
  • Changes span four documentation sections: content/integrations/, content/template-editor/, content/cli/, and content/tutorials/ (MEDIUM trigger)

Notes

  • The CodeBlock.tsx change is additive only (imports and registers xml and mjml languages for syntax highlighting) and does not modify existing behavior, but since it affects a shared component, verify that existing code blocks still render correctly
  • Confirm MJML syntax highlighting renders as expected in the new mjml.mdx page's code block
  • Verify all internal links to /integrations/email/mjml resolve correctly from the various cross-reference points (layouts, email-templates FAQ, html-partials, CLI docs, and tutorial)
  • Check that the new sidebar entry appears in the correct position within the Email integrations section
  • The new mjml.mdx file is missing a trailing newline (minor)
Open in Web View Automation 

Sent by Cursor Automation: Docs PR classifier

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Markdown link syntax is not processed inside JSX expressions in MDX,
so [MJML support](/integrations/email/mjml) rendered as literal text.
Replaced with an <a> tag to create a proper clickable link.

Co-authored-by: Chris Bell <chris@cjbell.co>
@cjbell cjbell merged commit 0e377a6 into main Mar 18, 2026
4 of 5 checks passed
@cjbell cjbell deleted the cb-mjml branch March 18, 2026 20:41
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.

4 participants