redirect tidy - #25115
redirect tidy#25115
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
Bot
added
area/compose
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
This PR removes stale alias redirects from front matter across 33 documentation files. All removals appear safe:
- Legacy Docker-for-Mac/Windows paths (, , , , , , etc.) — superseded by the top-level and aliases still retained in .
- Linux install intermediate paths (, etc.) — each page retains two remaining aliases covering the same targets.
- Previous-version release notes — pages have and are de-emphasized; removed paths are outdated versioned release note URLs.
- Compose intermediate paths (, , etc.) — legacy short URLs with no confirmed internal or external references within the docs corpus.
- Enterprise desktop paths — for a deprecated product, these removals are appropriate.
No internal documentation links were found using the removed aliases. No broken redirects detected.
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
This PR removes stale alias redirects from front matter across 33 documentation files. All removals appear safe:
- Legacy Docker-for-Mac/Windows paths (
/mac/,/win/,/windows/,/mackit/,/winkit/,/docker-for-mac/index/, etc.) — superseded by the top-level/docker-for-mac/and/docker-for-windows/aliases still retained insign-in.md. - Linux install intermediate paths (
/desktop/linux/install/ubuntu/, etc.) — each page retains two remaining aliases covering the same targets. - Previous-version release notes — pages have
sitemap: falseand are de-emphasized; removed paths are outdated versioned release note URLs. - Compose intermediate paths (
/compose/env/,/compose/envvars-precedence/, etc.) — legacy short URLs with no confirmed internal or external references within the docs corpus. - Enterprise desktop paths — for a deprecated product, these removals are appropriate.
No internal documentation links were found using the removed aliases. No broken redirects detected.
|
@dvdksn would the work in #25047 allow us to define some list for these that just redirects to the homepage? My thinking there is that having an explicit "this page no longer exists" (instead of a 404) could reduce some noise, and makes it clear to us that it's intentional that the URL is no longer there. (In general, I agree that most of these links are probably very old, so less likely to be hit) |
github-actions
Bot
added
the
area/get-started
|
@thaJeztah well, there is 410 Gone. Maybe it would be useful to distinguish 404 from 410. But there's some infra gymnastics needed for this I think. |
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
This PR removes stale and unused URL aliases (redirects) from front matter across 77 documentation files. The changes are well-scoped and the PR description notes that these were identified as unused via analytics.
Reviewed: 1 chunk, 1070 diff lines, 77 files
| - /desktop/features/dev-environments/set-up/ | ||
| - /desktop/features/dev-environments/share/ | ||
| - /desktop/features/dev-environments/dev-cli/ | ||
| - /desktop/dev-environments/ |
There was a problem hiding this comment.
[LOW] Dev Environments sub-page aliases removed — old deep-links will 404
The parent alias /desktop/features/dev-environments/ is still retained (good), but five aliases pointing to sub-pages and an older canonical path are removed:
/desktop/features/dev-environments/create-dev-env//desktop/features/dev-environments/set-up//desktop/features/dev-environments/share//desktop/features/dev-environments/dev-cli//desktop/dev-environments/(older canonical path without "features")
Since Dev Environments is retired, organic traffic to these paths should be low and declining. The PR description's "vvvvv old and unused" framing suggests usage analytics were checked, which is reassuring. The /desktop/dev-environments/ path (the shorter pre-"features" URL) is the one most likely to appear in older blog posts or support articles — worth a quick analytics check if not already done.
## Description `/docker-for-windows/troubleshoot/` currently returns a 404. This restores the alias on the Docker Desktop troubleshoot page. That alias was removed deliberately in #25115 ("redirect tidy"), along with seven others on the same page, as old and unused based on traffic analysis. I'm proposing to restore one of them because there's a live inbound link that traffic data wouldn't have shown as a working path: [VS Code's dev container tips and tricks page](https://code.visualstudio.com/docs/devcontainers/tips-and-tricks) links to `https://docs.docker.com/docker-for-windows/troubleshoot/#volumes`. Readers following that link land on a 404, which is how #26020 was reported. **Scope:** only `/docker-for-windows/troubleshoot/`. The other seven aliases removed in #25115 also 404 today, but I didn't find inbound links for them, so I left them alone. Happy to restore more if you'd prefer. **On the anchor:** `#volumes` no longer resolves — that content moved to the troubleshoot topics page. The alias lands readers on the troubleshoot overview, which links through to it. Still better than a 404. ## Related issues or tickets Fixes #26020 ## Reviews - [ ] Technical review - [ ] Editorial review - [ ] Product review Signed-off-by: Sunmin Lee <134378502+sunm2n@users.noreply.github.com>
5 participants
Description
As per Craigs number-crunching, this PR removes some vvvvv old and unused redirects 🧹
Related issues or tickets
Reviews