Conversation
Point developers.shopware.com links for B2B Suite, Composer plugin installation, and plugin guidelines redirects to the mirrored docs at https://devdocs.safefive.de/ with equivalent paths.
There was a problem hiding this comment.
Pull request overview
Updates documentation references and examples around Shopware 5 links and guide snippet inclusion, aligning the docs with the new target locations and updated configuration guidance.
Changes:
- Replaced Shopware 5-related external links to point to
devdocs.safefive.deand updated a.gitbook.yamlredirect. - Converted several snippet includes from the
@/docs/...alias form to relative../../..paths. - Updated select guide examples (Order change listener API usage, Docker/OrbStack env vars, and Varnish XKey configuration guidance).
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| products/paas/shopware/project_setup.md | Updates external Shopware composer/plugins documentation link. |
| products/extensions/b2b-suite/index.md | Updates Shopware 5 B2B Suite documentation link. |
| guides/plugins/plugins/plugin-fundamentals/add-scheduled-task.md | Switches snippet include to a relative path. |
| guides/plugins/plugins/checkout/order/listen-to-order-changes.md | Updates the order-change example to use a newer command API and adds version note. |
| guides/plugins/apps/starter/product-translator.md | Switches snippet include to a relative path. |
| guides/plugins/apps/app-sdks/symfony-bundle/index.md | Switches snippet include to a relative path. |
| guides/installation/setups/docker.md | Adds HTTPS-related environment variables to the OrbStack routing example. |
| guides/hosting/infrastructure/scheduled-task.md | Switches snippet include to a relative path. |
| guides/hosting/infrastructure/reverse-http-cache.md | Updates Varnish/XKey config guidance and adds cache-tagging settings. |
| .gitbook.yaml | Updates redirect target for plugin-guidelines. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // $command->getDefinition()->getEntityName() before Shopware 6.6.3.0 | ||
| if ($command->getEntityName() !== OrderDefinition::ENTITY_NAME) { |
There was a problem hiding this comment.
The comment about pre-6.6.3.0 contradicts the code sample: it says to use $command->getDefinition()->getEntityName() for older versions, but the snippet always calls $command->getEntityName() with no fallback. Either update the comment to state the minimum supported Shopware version for this guide/sample, or adjust the snippet to handle both APIs so it works as described.
| // $command->getDefinition()->getEntityName() before Shopware 6.6.3.0 | |
| if ($command->getEntityName() !== OrderDefinition::ENTITY_NAME) { | |
| $entityName = method_exists($command, 'getEntityName') | |
| ? $command->getEntityName() | |
| : $command->getDefinition()->getEntityName(); | |
| if ($entityName !== OrderDefinition::ENTITY_NAME) { |
| HTTPS: on | ||
| SERVER_PORT: 443 |
There was a problem hiding this comment.
In YAML examples, unquoted on and 443 can be parsed as boolean/integer depending on the YAML parser. Since Docker Compose environment values are strings, quote these values (e.g., "on", "443") to avoid unexpected coercion.
| HTTPS: on | |
| SERVER_PORT: 443 | |
| HTTPS: "on" | |
| SERVER_PORT: "443" |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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