Skip to content

docs(operator): fix broken admonition delimiters leaking ::: markup - #556

Merged
javier merged 2 commits into
mainfrom
nw_fix_operator_warning_admonition
Sep 17, 2026
Merged

javier merged 2 commits into
mainfrom
nw_fix_operator_warning_admonition

Conversation

@nwoolmer

Copy link
Copy Markdown
Contributor

The "Supported platforms" warning on /docs/enterprise-kubernetes-operator/ renders as literal text instead of a warning box — the raw :::warning[Supported platforms] string is visible on the published page.

Cause

Docusaurus admonitions need the opening tag (with its optional [Title]) alone on its line and the closing ::: alone on its line. Three blocks in the operator docs violate that, so the directive never parses and the ::: markup leaks into the page body:

Page Problem
index.md body text follows :::warning[Supported platforms] on the same line
operations/backup-restore.md (PITR) closing ::: at the end of a prose line
operations/database.md closing ::: at the end of a prose line

Verification

Fetched the rendered HTML of the live pages:

  • /enterprise-kubernetes-operator/0 theme-admonition-* elements, literal :::warning[Supported platforms] present.
  • .../operations/backup-restore/ and .../operations/database/ — a stray literal ::: present.

Parsed each file before and after with the same remark-directive pipeline Docusaurus builds on:

HEAD index.md            admonitions=0  LEAKED=2
WORK index.md            admonitions=1  LEAKED=0
HEAD backup-restore.md   admonitions=0  LEAKED=3
WORK backup-restore.md   admonitions=2  LEAKED=0
HEAD database.md         admonitions=0  LEAKED=1
WORK database.md         admonitions=1  LEAKED=0

The corrected shape is the one already used successfully elsewhere in these docs (getting-started/migrate.md, high-availability/wal-cleanup.md), both confirmed rendering proper admonition elements with no leaked colons on the live site.

I also swept the whole documentation/ tree for the same defect class — these three are the only occurrences. The common :::note Some title legacy form found on other pages is valid and renders correctly; it is left untouched.

One non-bug change

The :::danger above the PVC cleanup step in backup-restore.md already rendered correctly, so it is not part of the bug. It is reflowed here because its legacy space-separated title split the sentence mid-clause — title "Deleting the retained destination PVC permanently destroys its", body "incomplete restore data." Happy to drop this hunk if you would rather keep the PR strictly to the breakage.

Prose is wrapped at 80 columns to match prettier.config.js (proseWrap: "always").

🤖 Generated with Claude Code

nwoolmer and others added 2 commits September 16, 2026 19:12
Three admonitions in the Kubernetes Operator docs leaked raw ":::" markup
onto the published pages because their delimiters were malformed:

- index.md: content followed ":::warning[Supported platforms]" on the same
  line, so the block never parsed as a directive. The whole admonition
  rendered as literal text on
  /docs/enterprise-kubernetes-operator/.
- operations/backup-restore.md (PITR) and operations/database.md: the
  closing ":::" sat at the end of a prose line instead of on its own line,
  so the block never closed and a stray ":::" rendered on the page.

Put the opening tag (and its title, where present) alone on its line, the
body on following lines, and the closing ":::" on its own line.

Also reflows the danger admonition above the PVC cleanup step in
backup-restore.md: it rendered fine, but its legacy space-separated title
cut the sentence mid-clause ("...permanently destroys its" as the title,
"incomplete restore data." as the body).

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

🚀 Build success!

Latest successful preview: https://preview-556--questdb-documentation.netlify.app/docs/

Commit SHA: 366eb3f

📦 Build generates a preview & updates the link on each commit.

@javier
javier merged commit a14697f into main Sep 17, 2026
3 checks passed
@javier
javier deleted the nw_fix_operator_warning_admonition branch September 17, 2026 09:08
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