Fix 404 redirects for documentation site#177
Merged
Merged
Conversation
- Fix /blog/* redirect to point to /connect/home/ instead of non-existent /connect/about/ - Add version root redirects for 23.3, 24.1, 24.2, 24.3, 25.1, 25.2, 25.3 to their streaming component homes - Add catch-all redirects for old streaming version paths (23.1, 23.2, 22.x, 21.11) to streaming/current
✅ Deploy Preview for redpanda-documentation ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
- Update all playbooks to use latest UI bundle release URL - Enable announcement banner in antora-playbook.yml and preview-antora-playbook.yml - local-antora-playbook.yml already had announcements enabled
micheleRP
approved these changes
May 27, 2026
Contributor
micheleRP
left a comment
There was a problem hiding this comment.
Review
Approving — the redirect targets are correct, the new catch-alls cover real 404 sources, and the /blog/* → /connect/home/ fix addresses the highest-volume case. Verified that /connect/home/ exists (lines 79–80 and 95–96 of netlify.toml already reference it) and that the new /streaming/X/* catch-alls don't duplicate the existing bare /X/* redirects further down.
Heads-up (not blocking)
- Netlify deploy preview is failing on commit
1bcc5ff(deploy log). Worth a quick look before merging in case the build failure is related to thelatestUI bundle URL change or the re-enabledserve-markdownedge function. - UI bundle URL moved from a pinned release (
v3.0.0-beta.23) toreleases/latest/download/ui-bundle.zip.latestcurrently resolves tov3.0.0(released yesterday). This is a workflow shift worth being aware of — future docs-ui releases will deploy without an explicit docs-site PR. Combined withsnapshot: true, every build will pull the newest bundle. Fine if intentional; just calling it out so it's not a surprise the first time a UI release breaks something. - Description nit — the body says catch-all redirects were added for "23.1, 23.2, 22.x, 21.11"; the diff actually adds six specific versions (21.11, 22.1, 22.2, 22.3, 23.1, 23.2). Worth updating for accuracy.
- Optional follow-up: bare version roots are redirected for supported versions (
/25.3→/streaming/25.3/home/) but not for EOL versions (/22.1,/21.11, etc.). If/24.1was 404ing,/22.1likely is too. Easy follow-up if it shows up in the next round of friction data.
What works well
- The split of redirect targets is sensible — supported versions go to their version-specific home, EOL versions collapse to
/streaming/current/. - 301 status code is right for permanent redirects.
- The new
/streaming/<v>/*catch-alls fill in a real gap (existing redirects only covered the bare/<v>/*paths). - The
/connect/about/→/connect/home/fix targets a legitimately broken redirect.
Add bare root redirects for EOL versions (23.2, 23.1, 22.3, 22.2, 22.1, 21.11) to streaming/current/home/, matching the pattern used for supported versions.
Fetch and inject chat-panel-bump.html widget to enable the Ask AI drawer on Bump.sh API documentation pages. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Offset chat panel top to account for fixed navbar height - Add dark mode styles using html[data-theme="dark"] selector (Bump pages use this attribute, not .dark-theme class) - Ensure chat panel icons are visible via stroke/fill fixes Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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
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
This PR fixes multiple 404 redirect issues that were causing a high volume of errors on the documentation site.
Changes
1. Fixed
/connect/about/redirect (High Priority)/blog/*redirect pointed to/connect/about/, which doesn't exist/connect/home/, which is the actual Connect component home page2. Added version root redirects
/23.3/,/24.1/, etc. had no redirects and returned 404s3. Enhanced old version coverage
/streaming/23.1/*only had specific redirects, not catch-alls/streaming/23.1/*,/streaming/23.2/*,/streaming/22.x/*,/streaming/21.11/*to/streaming/current/Testing
After deployment, verify:
/connect/aboutand/connect/about/redirect to/connect/home//23.3,/24.1,/25.3redirect to their streaming homes/streaming/23.1/manage/cluster-maintenance/cluster-balancing/redirect to their current equivalentsFiles Modified
netlify.toml- Added 72 lines of redirect rules