Skip to content

Remove duplicated pagination #8049

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 16, 2025

Conversation

Prokyonn
Copy link
Member

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
License MIT

What's in this PR?

Removes duplicated pagination config on the query builder in the SmartContentProviders

Why?

Pagination should only be set once 😄

@Prokyonn Prokyonn added the Bug Error or unexpected behavior of already existing functionality label Jul 16, 2025
@Prokyonn Prokyonn force-pushed the bugfix/smart-content-pagination branch from a738e69 to 4838364 Compare July 16, 2025 13:34
@Prokyonn Prokyonn force-pushed the bugfix/smart-content-pagination branch from 4838364 to 7ffac87 Compare July 16, 2025 13:41
@Prokyonn Prokyonn enabled auto-merge (squash) July 16, 2025 13:41
@Prokyonn Prokyonn disabled auto-merge July 16, 2025 13:51
@Prokyonn Prokyonn merged commit 581163e into sulu:3.0 Jul 16, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Error or unexpected behavior of already existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant