Skip to content

Fix mandatory condition for page tree route #7842

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
Mar 11, 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?

Adds property metadata to the PageTreeRouteContentType.

Why?

The mandatory condition of the page_tree_route content type did not work properly.

@Prokyonn Prokyonn force-pushed the bugfix/page-tree-route branch from da4c7ab to f616221 Compare March 10, 2025 19:32
@alexander-schranz alexander-schranz added the Bug Error or unexpected behavior of already existing functionality label Mar 10, 2025
@alexander-schranz
Copy link
Member

Think we should backport this also to 2.5.

@Prokyonn Prokyonn changed the base branch from 2.6 to 2.5 March 11, 2025 07:11
@Prokyonn Prokyonn force-pushed the bugfix/page-tree-route branch from f616221 to 435841d Compare March 11, 2025 07:12
@alexander-schranz alexander-schranz enabled auto-merge (squash) March 11, 2025 07:16
@Prokyonn Prokyonn disabled auto-merge March 11, 2025 08:49
@Prokyonn Prokyonn merged commit 0188a1c into sulu:2.5 Mar 11, 2025
8 of 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.

2 participants