Skip to content

docs: document ClassMoved compatibility attribute - #2523

Open
Jonas Elfering (keulinho) wants to merge 2 commits into
mainfrom
codex/document-class-moved
Open

Jonas Elfering (keulinho) wants to merge 2 commits into
mainfrom
codex/document-class-moved

Conversation

@keulinho

@keulinho Jonas Elfering (keulinho) commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add #[ClassMoved] to the BC-change attribute reference.
  • Document its runtime alias semantics for call sites and extending classes.
  • Explain how extension call sites, subclasses, and class-name service IDs migrate to the canonical name.
  • Replace the previous recommendation to duplicate a class when changing its namespace.

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

This documentation describes the ClassMoved attribute introduced by shopware/shopware#20525 and should be merged after that implementation.

Copilot AI lite review requested due to automatic review settings September 17, 2026 12:36
@shopware-dev-docs-connector

shopware-dev-docs-connector Bot commented Sep 17, 2026

Copy link
Copy Markdown

Developer Docs healthcheck

Status: Completed with success.
Repository: shopware/docs
Commit: 5c7415c
Preview: https://developer-documentation-18jrznkrq-shopware-frontends.vercel.app
Workflow run: #5324

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.

🟢 Approval recommended

No unresolved blocking issues were identified.

Pull request overview

Documents the ClassMoved backward-compatibility attribute and its class/service alias lifecycle.

Changes:

  • Adds ClassMoved guidance to the compatibility documentation.
  • Replaces namespace-duplication advice with alias-based migration steps.
  • Adds ClassMoved to the spellcheck wordlist.
File summaries
File Description
resources/guidelines/code/backward-compatibility.md Documents ClassMoved usage, runtime behavior, and migration lifecycle
.wordlist.txt Adds ClassMoved to the allowed spellcheck terms
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants