Skip to content

fix(docs): MDX-safe TODO placeholders in journeydoc tutorials#432

Merged
rubenvdlinde merged 1 commit into
developmentfrom
fix/docs-mdx-todo-placeholders
May 12, 2026
Merged

fix(docs): MDX-safe TODO placeholders in journeydoc tutorials#432
rubenvdlinde merged 1 commit into
developmentfrom
fix/docs-mdx-todo-placeholders

Conversation

@rubenvdlinde
Copy link
Copy Markdown
Contributor

The journeydoc bootstrap left bare {{TODO: ...}} text in docs/tutorials/**/*.md; Docusaurus 3 parses .md as MDX so {...} is read as a JS expression and npm run build fails. Wrapped them in HTML comments (<!-- {{TODO: ...}} -->) so they stay as markers, don't render, and are MDX-safe. Docs build now passes.

@rubenvdlinde rubenvdlinde requested a review from Rem-Dam as a code owner May 12, 2026 22:08
@rubenvdlinde rubenvdlinde merged commit e8fa060 into development May 12, 2026
16 checks passed
@rubenvdlinde rubenvdlinde deleted the fix/docs-mdx-todo-placeholders branch May 12, 2026 22:09
rubenvdlinde added a commit that referenced this pull request May 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/procest @ e2d4c85

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
composer ✅ 100/100
npm ✅ 420/420
PHPUnit
Newman ⏭️
Playwright

Coverage: 0% (0/81 statements)

Spec coverage: 7% (30 tests / 456 specs)


Quality workflow — 2026-05-12 22:24 UTC

Download the full PDF report from the workflow artifacts.

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.

1 participant