Skip to content

Added support for including system content type groups in ContentTypeController #208

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.

Sign up for GitHub

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

Jump to bottom
Open
mikadamczyk wants to merge 11 commits into
base: 4.6
Choose a base branch
from
Open

Added support for including system content type groups in ContentTypeController #208

mikadamczyk wants to merge 11 commits into from

Conversation

@mikadamczyk
Copy link
Contributor

@mikadamczyk mikadamczyk commented Nov 28, 2025 •
edited
Loading

🎫 Issue N/A

Related PRs:

Description:

  • Exposed includeSystem query parameter on /content/typegroups and forwarded it to loadContentTypeGroups, enabling optional inclusion of system content type groups while preserving the default non-system behaviour.
  • Documented the new query parameter in the OpenAPI metadata.
  • Added unit tests for the ContentType controller to cover default filtering and the includeSystem=true path.

For QA:

Documentation:

@mikadamczyk mikadamczyk self-assigned this Nov 28, 2025
@mikadamczyk mikadamczyk mentioned this pull request Nov 28, 2025
Open
alongosz
alongosz requested changes Nov 28, 2025
View reviewed changes
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 1, 2025 •
edited
Loading

Quality Gate Passed Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@mikadamczyk mikadamczyk requested a review from alongosz December 2, 2025 10:15
@mikadamczyk mikadamczyk marked this pull request as ready for review December 4, 2025 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alongosz alongosz Awaiting requested review from alongosz

Assignees

@mikadamczyk mikadamczyk

@Steveb-p Steveb-p

@alongosz alongosz

@tischsoic tischsoic

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mikadamczyk @alongosz @Steveb-p @tischsoic