Skip to content

Update smart content documentation#889

Open
Prokyonn wants to merge 3 commits intosulu:3.xfrom
Prokyonn:enhancement/smart-content
Open

Update smart content documentation#889
Prokyonn wants to merge 3 commits intosulu:3.xfrom
Prokyonn:enhancement/smart-content

Conversation

@Prokyonn
Copy link
Member

Q A
Fixed tickets fixes #issuenum
Related PRs sulu/sulu#prnum
License MIT

What's in this PR?

Updates the documentation to be compatible with Sulu 3.0

Why?

The documentation still had wrong Sulu 2.6 content.

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

Updates Smart Content documentation to align with Sulu 3.0 behavior and APIs, replacing outdated Sulu 2.6 guidance.

Changes:

  • Updates Smart Content property type reference (new pagination/view fields, sorting changes, new provider notes, and updated examples).
  • Updates SmartContentProvider cookbook registration tag for Sulu 3.0.
  • Updates book pagination description to include total and maxPage.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
reference/property-types/smart_content.rst Refreshes Smart Content reference (warning about duplicates, sortBys, pagination fields, entity property access, articles provider, example mappings/templates).
cookbook/smart-content-data-provider.rst Updates service tag name/attributes for registering smart content providers in Sulu 3.
book/smart-content.rst Updates pagination narrative to mention total and maxPage.

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

@Prokyonn Prokyonn force-pushed the enhancement/smart-content branch 2 times, most recently from 05ae1c8 to 7ad5def Compare February 11, 2026 13:41
@Prokyonn Prokyonn force-pushed the enhancement/smart-content branch from 7ad5def to dd95523 Compare February 18, 2026 08:14
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