DOC-7104: Fix link issues found by review on the render-hook migration's sibling PRs - #4103
Merged
Merged
Conversation
…n's sibling PRs Follow-up cleanup on already-merged units, mirroring what human review caught on the still-open DOC-7104 PRs for the frozen version snapshots: - _index.md, references/cli-utilities/redis-cli/_index.md, references/compatibility/commands/{cluster,connection,generic,pub-sub, scripting,server,transactions}.md, security/access-control/ redis-acl-overview.md: bare /commands[?query] links missing the /content/ prefix used everywhere else in this migration. Confirmed via rendered HTML that both forms produce an identical href (render-link.html strips the /content prefix before its own GetPage lookup, and Hugo auto-generates a section page for /commands even with no _index.md on disk) -- a form/consistency fix, not a functional one. - new-features-redis-enterprise.md: applied the same /content/ prefix to the 6 known pre-existing dead relref targets in this draft: true page, for consistency. Cosmetic only -- none of the 6 targets exist regardless of prefix. - databases/import-export/flush.md: the port-configurations link's anchor was `#https://docs.redis.com/latest/rs/networking/port-configurations# ports-and-port-ranges-used-by-redis-enterprise-software` -- a stray old absolute URL concatenated into the fragment, present in the source relref shortcode before this migration touched it. Replaced with the real heading slug (`## Ports and port ranges used by Redis Software`). Verified via build/diff_rendered_hrefs.py: 2,404 pages compared, exactly 1 href changed (flush.md's corrected anchor, the intended fix) -- every /commands-prefix fix produced 0 href diff, confirming those are purely cosmetic. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
Contributor
This was referenced Sep 24, 2026
Contributor
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at 8c8f8a6 |
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
Follow-up cleanup on already-merged DOC-7104 units, applying the same fixes human review caught on the still-open sibling PRs for the frozen version snapshots (#4093-#4100):
/commands[?query]links missing the/content/prefix (_index.md,references/cli-utilities/redis-cli/_index.md, 7 files inreferences/compatibility/commands/,security/access-control/redis-acl-overview.md). Confirmed via rendered HTML that both forms produce an identical href — a form/consistency fix, not a functional one.new-features-redis-enterprise.md: applied the same/content/prefix to the 6 known pre-existing dead relref targets in thisdraft: truepage, for consistency. Cosmetic only — none of the 6 targets exist regardless of prefix.databases/import-export/flush.md: the port-configurations link's anchor was a stray old absolute URL concatenated into the fragment (present in the source relref shortcode before this migration touched it). Replaced with the real heading slug.Test plan
build/diff_rendered_hrefs.py: 2,404 pages compared, exactly 1 href changed (flush.md's corrected anchor, the intended fix) — every/commands-prefix fix produced 0 href diff, confirming those are cosmetic only.🤖 Generated with Claude Code
Note
Low Risk
Documentation-only link path and anchor corrections with no application or security impact; one href change intended on flush.md.
Overview
DOC-7104 follow-up that aligns Redis Software docs with the render-hook link style after review on sibling migration PRs.
Most edits are form-only: bare
/commandsand/operate/...targets become/content/commandsand/content/operate/...on the RS index,redis-clireference, seven command-compatibility pages, and the Redis ACL overview. Rendered hrefs for those/commandsupdates are unchanged; this is consistency, not behavior.The only functional link fix is in
databases/import-export/flush.md: the port-configurations note now uses the correct heading fragment (#ports-and-port-ranges-used-by-redis-software) instead of a broken anchor that embedded an old docs URL.new-features-redis-enterprise.md(draft) gets the same/content/prefix on six internal links for consistency; those targets were already missing.Reviewed by Cursor Bugbot for commit 8c8f8a6. Bugbot is set up for automated code reviews on this repo. Configure here.