02: Delete the redirect map and the redirects plugin configuration - #3383
Merged
Conversation
Remove all 480 entries of the mkdocs-redirects redirect_maps block from plugins.yml, along with the redirects plugin registration itself, and drop the now-unused mkdocs-redirects dependency from requirements.txt. The same redirects remain live on the maintained on-premise documentation branches, which continue to serve those URLs. Doing this first means no later conversion batch has to decide whether a page it deleted needs a redirect entry. build_package_docs.py still reads redirect_maps from plugins.yml and degrades to an empty map when the block is absent, so package builds and their tests are unaffected. No documentation page is touched. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Preview of modified files: no change to preview. |
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
1 participant
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.
Ticket 02 of the Cohesivo SaaS conversion. Base:
saas-batch-3.Deletes the legacy URL redirect map so no later conversion batch has to decide
whether a page it deleted needs a redirect entry — the mechanism is gone.
Changes
plugins.yml: removes theredirectsplugin registration and all 480redirect entries (-511 lines).
requirements.txt: dropsmkdocs-redirects==1.2.2, now unused.No file under
docs/is touched, which the ticket requires.Accepted consequence
Inbound links to those 480 legacy URLs no longer resolve on this site. This is
deliberate: the same redirects remain live on the maintained on-premise
documentation branches, which continue to serve those URLs. No new redirects are
added for pages this conversion deletes.
Worth noting the ticket's estimate was off in a reassuring direction — it
expected "roughly 400 of the deleted entries point at pages that still exist",
but none of the 480 sources exists as a real file under
docs/. Every sourceis a legacy path, so no live page was shadowing a redirect.
Verification
run locally over all 4158 built pages instead — 0 absolute self-links resolve
to a removed redirect source.
Follow-up for the tooling batch (ticket 21)
.github/pull_request_template.md:14still carries a "Redirects coverremoved/moved pages" checklist item, and
AGENTS.mdstill documents the plugin.Both are repository tooling rather than content, so they belong in that pull
request, not this one.
Note:
build_package_docs.pyreadsredirect_mapsout ofplugins.ymlandreturns an empty map when the plugin is absent, so it degrades cleanly. The
package builder now resolves zero redirects — a real behaviour change, and the
correct one.
🤖 Generated with Claude Code