DOC-7118 unit 5: convert rc subscriptions/ to render hooks - #4123
Merged
Merged
Conversation
Migrate relref shortcode links and note/warning/tip/info callout shortcodes to the DOC-6909 render-hook equivalents (plain Markdown links + `> [!TYPE]` blockquotes) across content/operate/rc/subscriptions/. 88 relref links and 17 callouts converted via build/migrate_shortcode_links.py. Two list-nested callouts (aws-console.md, set-maintenance-windows.md) hit the known indentation-loss bug in the callout-to-blockquote conversion; hand-fixed so every continuation/closing line matches the header's indent, closing with a bare `>` at that indent, per the established convention (verified against content/operate/rs/databases/recover.md). Verified: 0 remaining relref/callout shortcodes, 0 uncanonicalized-link findings via check_uncanonicalized_links.py across all of content/operate/rc/, and 0 href diffs between a real Hugo build of origin/main vs this branch (full site: 7235 vs 7235 pages; operate/rc scope: 277 vs 277 pages). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
Contributor
Contributor
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at 91787ea |
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
content/operate/rc/subscriptions/(16 files) offrelrefshortcode links andnote/warning/tip/infocallout shortcodes to the DOC-6909 render-hook equivalents (plain Markdown links +> [!TYPE]blockquotes), matching the pattern established across every othercontent/operate/andcontent/develop/directory in prior units. Units 1-4 of this ticket are open as PRs DOC-7118 unit 1: convert rc top-level/accounts/billing/logs-reports to render hooks #4118, DOC-7118 unit 2: convert rc api/cloud-integrations/context-engine to render hooks #4119, DOC-7118 unit 4: convert rc security/ to render hooks #4120, DOC-7118 unit 3: convert rc databases/ to render hooks #4121.note, 7warning, 1tip, plus the rest already-plain).bring-your-own-cloud/iam-resources/aws-console.md,maintenance/set-maintenance-windows.md) hit the known indentation-loss bug in the callout-to-blockquote conversion, which detaches the blockquote's continuation/closing lines from their parent<li>and leaves a spurious trailing-whitespace>line. Hand-fixed both so every continuation/closing line matches the header's indent, closing with a bare>at that indent, matching the established convention (verified againstcontent/operate/rs/databases/recover.md). Confirmed correct<li>nesting in the rendered HTML forset-maintenance-windows.md;aws-console.md's canonical page doesn't render its full content in either the before or after build in this environment due to a pre-existing, unrelated network-dependent shortcode (external-json/GetRemote) limitation — confirmed identical in both builds, so not a regression from this change.{{< relref ... >}}suffix(no separating slash) instances, no unusual closing-tag spacing ({{< /warning >}}etc. all matched the converter's regex correctly in this unit's files).Verification
build/check_uncanonicalized_links.py --fix content/operate/rc/: 185 files scanned,FIXABLE=0 MOUNT_ONLY=0 DEAD=0.build/diff_rendered_hrefs.py) oforigin/mainvs this branch, both withnpm installrun and a workingpostcsspipeline (fixed a sandboxed-npm-cache permission issue that had silently broken the CSS minification pipeline in earlier verification attempts):operate/rcscope: 277 vs 277 pages, 0 href diffs.Test plan
python3 build/migrate_shortcode_links.py all <file>run on all 16 files, each confirmed changed via diff against pre-conversion state.build/check_uncanonicalized_links.py --fix content/operate/rc/— 0 findings.origin/main(before) and this branch (after),build/diff_rendered_hrefs.py— 0 href diffs, both full-site andoperate/rc-scoped.git status/git diff --statconfirm exactly the 16 target files changed, nothing else.Refs DOC-7118.
🤖 Generated with Claude Code
Note
Low Risk
Documentation-only shortcode-to-Markdown migration with verified zero rendered href diffs; no application or runtime code changes.
Overview
Unit 5 of DOC-7118 updates 16 pages under
content/operate/rc/subscriptions/to match the render-hook pattern used elsewhere inoperate/anddevelop/.Internal links: Hugo
relrefshortcodes are replaced with plain Markdown links using canonical/content/operate/...paths (including_index.md,.md, and fragment anchors). Callouts:note,warning, andtipshortcodes become GitHub-style blockquotes (> [!NOTE],> [!WARNING],> [!TIP]).Two list-nested callouts in
bring-your-own-cloud/iam-resources/aws-console.mdandmaintenance/set-maintenance-windows.mdwere hand-adjusted so blockquote continuation lines stay aligned with their parent list items after conversion.Prose and behavior are unchanged; this is a linking/callout format migration only.
Reviewed by Cursor Bugbot for commit 91787ea. Bugbot is set up for automated code reviews on this repo. Configure here.