Skip to content

Conversation

@nickygerritsen
Copy link
Contributor

This showed section in the path instead of block.

This showed `section` in the path instead of `block`.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes incorrect path documentation for CMS blocks by changing "section" to "block" in the directory path. While the directory change is correct, the filename itself appears to have an additional issue with redundant "block" text.

  • Changed the incorrect directory from section/ to block/ in the example path

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Isengo1989 Isengo1989 added the Storefront ct-storefront label Jan 5, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* **Extension**: `.html.twig`

**Shopware is expecting the prefix as part of the full filename in `src/Storefront/Resources/views/storefront/section/cms-section-block-container.html.twig`.**
**Shopware is expecting the prefix as part of the full filename in `src/Storefront/Resources/views/storefront/block/cms-block-container.html.twig`.**
Copy link
Contributor

@bschulzebaek bschulzebaek Jan 6, 2026

Choose a reason for hiding this comment

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

Actually this path should be correct: it refers to the place where blocks are loaded to provide a bit more technical context. We could rephrase it a bit:

[!NOTE]
Block components are loaded in `src/Storefront/Resources/views/storefront/section/cms-section-block-container.html.twig`. This loader expect the following format for block components: `cms-block-image-text-reversed.html.twig`.

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

Labels

Storefront ct-storefront

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants