DOC-7118 unit 3: convert rc databases/ to render hooks - #4121
Conversation
Migrate relref shortcode links and note/warning/tip/info callout shortcodes to plain Markdown links and > [!NOTE] blockquotes across content/operate/rc/databases/ (30 of 31 listed files changed; the 31st, configuration/_index.md, had nothing to convert). Hand-fixed 8 callout instances nested inside list items where the converter dropped the continuation-line indent and left a spurious trailing '>' artifact at the old closing-tag position, matching the established rs/ convention (4/8-space indent on every blockquote line, terminating ' >' line). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at 9394973 |
| [Learn more about | ||
| synchronization for | ||
| each supported data type]({{< relref "/operate/rc/databases/active-active/develop/data-types/" >}}) and [how to develop applications]({{< relref "/operate/rc/databases/active-active/develop/develop-for-aa" >}}) with them on Redis Cloud. No newline at end of file | ||
| each supported data type](/operate/rc/databases/active-active/develop/data-types/) and [how to develop applications](/operate/rc/databases/active-active/develop/develop-for-aa) with them on Redis Cloud. No newline at end of file |
There was a problem hiding this comment.
I'm surprised these links work (no .md extension). 🤔
There was a problem hiding this comment.
Both targets (active-active/develop/data-types/ and develop-for-aa) are mount-only — they're a Hugo module mount of content/operate/rs/databases/active-active/develop/ (see config.toml), so there's no real file under content/operate/rc/ for the converter to resolve to a /content/<path>.md target. build/check_uncanonicalized_links.py correctly reports these as MOUNT_ONLY and leaves them as bare paths rather than guessing — same rationale as its own mount-avoidance for _find_content_file. Confirmed functionally fine: the site-wide href diff for this exact page came back 0 changes, so the rendered link target is byte-identical to the pre-conversion relref output. Every other link on this page that resolves to a real content file did get the full /content/...md treatment.
dwdougherty
left a comment
dwdougherty
left a comment
There was a problem hiding this comment.
Just one comment (nothing to fix). LGTM.
2 participants
Summary
Unit 3 of DOC-7118: migrates
relrefshortcode links andnote/warning/tip/infocallout shortcodes to the DOC-6909 render-hook equivalents (plain Markdown links +> [!NOTE]blockquotes) acrosscontent/operate/rc/databases/. Wave 1 of this ticket (units 1-2) is in PRs #4118 and #4119; this is unit 3.content/operate/rc/databases/configuration/_index.mdhad norelref/callout syntax to convert, so it's untouched).relrefoccurrences and 35 callout shortcode blocks, viabuild/migrate_shortcode_links.py all <files>.>artifact at the old closing-tag position. Re-indented to match the establishedrs/convention (4- or 8-space indent on every blockquote line, terminating>line), then verified against a real Hugo build that the affected pages'<li>nesting still renders correctly.build/check_uncanonicalized_links.py --fixrun scoped tocontent/operate/rc/databases/: fixed 2 bare/commandslinks (now/content/commands), confirmed 0 FIXABLE remaining. The only other findings are 3 expectedMOUNT_ONLYhits underactive-active/develop/_index.mdandactive-active/_index.mdpointing at the rs/-mounteddevelop-for-aa/data-types//app-failover-active-activefiles — those are physically incontent/operate/rs/databases/active-active/develop/, already converted via DOC-7104, and out of scope here per the ticket's mount scoping note.origin/main(before) and this branch (after), diffed withbuild/diff_rendered_hrefs.py ... operate/rc: 0 href diffs across all 278operate/rcpages compared.Note: to get a clean full-site build in this environment, the 5 unrelated pages under
content/develop/clients/{nodejs,redis-py,ruby,hiredis}that use thejupyter-exampleshortcode had to have that shortcode call temporarily stripped in both the before and after checkouts (the shortcode'sreadFileneeds a generatedexamples/directory that isn't present in a fresh checkout) — this was reverted before committing and touches no file in this PR's diff.Test plan
build/migrate_shortcode_links.py all <files>run on all 31 unit files; diffed each against pre-conversion state>}}-without-slash relrefs, malformed closing-tag spacing) before and after conversionbuild/check_uncanonicalized_links.py --fixscoped to this unit: 0 FIXABLE remainingbuild/diff_rendered_hrefs.py ... operate/rc: 0 href diffsRef: DOC-7118
🤖 Generated with Claude Code
Note
Low Risk
Documentation-only link and callout syntax changes; PR verification reports zero rendered href diffs for operate/rc after full Hugo builds.
Overview
Unit 3 (DOC-7118) updates Redis Cloud databases docs under
content/operate/rc/databases/for DOC-6909 render hooks: Hugorelreflinks become canonical Markdown paths (mostly/content/...), andnote/warning/tip-style shortcodes become GitHub-flavored> [!NOTE]/> [!WARNING]blockquotes.Roughly 30 pages are touched (indexes, Active-Active, create/view/migrate/monitor, configuration, connect, version management, etc.). Prose and Hugo embeds (
{{< image >}},{{< embed-md >}}, multitabs) are unchanged. A few list-nested callouts were re-indented manually so list markup still renders correctly. A small number of links to rs-mounted Active-Active develop paths stay as site paths (e.g./operate/rc/...) rather than/content/..., per existing mount scoping.Reviewed by Cursor Bugbot for commit 9394973. Bugbot is set up for automated code reviews on this repo. Configure here.