Skip to content

fix: fix 404 links#2330

Merged
Micha Hobert (Isengo1989) merged 2 commits into
shopware:mainfrom
M-arcus:fix-404
Jun 16, 2026
Merged

fix: fix 404 links#2330
Micha Hobert (Isengo1989) merged 2 commits into
shopware:mainfrom
M-arcus:fix-404

Conversation

@M-arcus

Copy link
Copy Markdown
Contributor

Summary

  • fixes 404 links

Related links

Checklist

  • I reviewed affected links, code samples, and cross-references, including PageRef references where relevant.
  • I added or updated redirects in .gitbook.yaml if pages were moved, renamed, or deleted.
  • I updated .wordlist.txt (and sorted it) if spellcheck flags new legitimate terms.
  • Any required dependent changes in downstream modules have already been merged and published.
  • This pull request is ready for review.

Notes

Copilot AI review requested due to automatic review settings June 15, 2026 08:58
@M-arcus

Copy link
Copy Markdown
Contributor Author

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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-server location.
  • Fix an escaped MDN anchor (\##) in a PageRef URL.

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.

Comment thread resources/references/adr/2025-10-28-changelog-release-info-process.md Outdated
Comment thread resources/guidelines/code/core/6.5-new-php-language-features.md Outdated
Comment thread resources/references/adr/2024-07-30-add-telemetry-abstraction-layer.md Outdated
@Isengo1989

Micha Hobert (Isengo1989) commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

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 🙂

@M-arcus

Copy link
Copy Markdown
Contributor Author

Micha Hobert (@Isengo1989)

[...] 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.

@Isengo1989 Micha Hobert (Isengo1989) merged commit 0d45f58 into shopware:main Jun 16, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Bugfixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants