fix: fix 404 links#2330
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates documentation/ADR references to reflect moved content and fix broken/escaped links so readers can navigate to the right resources.
Changes:
- Replace outdated relative paths with GitHub or updated in-repo links.
- Update MCP Server guide links to the new
products/tools/mcp-serverlocation. - Fix an escaped MDN anchor (
\#→#) in aPageRefURL.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| resources/references/adr/2025-10-28-changelog-release-info-process.md | Removes a relative link and replaces it with a plain-text reference to an internal guide. |
| resources/references/adr/2024-07-30-add-telemetry-abstraction-layer.md | Switches telemetry README reference to a GitHub URL. |
| resources/references/adr/2021-05-28-introduce-eslint-on-vue-admin.md | Switches internal relative links to GitHub URLs for plugin and VSCode settings. |
| resources/guidelines/code/core/6.5-new-php-language-features.md | Replaces an ADR link with a non-linked “historical” reference. |
| resources/guidelines/code/backward-compatibility.md | Updates feature-flag and changelog references to new local doc paths. |
| guides/plugins/plugins/storefront/howto/use-media-thumbnails.md | Fixes MDN URL anchor escaping in PageRef. |
| guides/plugins/plugins/mcp-server.md | Updates MCP Server documentation links to products/tools. |
| guides/plugins/apps/mcp-server.md | Updates MCP Server documentation links + fixes app-scripts relative paths. |
| guides/development/extensions/index.md | Updates MCP Server index link to products/tools. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Marcus Müller (@M-arcus), thx for the PR 👍 We do not fix links from code/core/guidelines and ADR here, as they are imported from shopware/shopware. https://github.com/shopware/docs#synced-files I will create an issue on the matter and we will see what is actually fixable and what should stay. Please remove the changes from the PR, and I will merge it ASAP. You are free to create a PR on shopware/shopware to fix those as well of course 🙂 |
I reverted code/core/guidelines and ADR changes. |
87dd90b to
6cae856
Compare
Summary
Related links
Checklist
PageRefreferences where relevant..gitbook.yamlif pages were moved, renamed, or deleted..wordlist.txt(and sorted it) if spellcheck flags new legitimate terms.Notes