Skip to content

make symfony/security-acl integration optional #8369

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
Merged
dmaicher merged 1 commit into from
Jan 5, 2026
Merged

make symfony/security-acl integration optional #8369

dmaicher merged 1 commit into from
Jan 5, 2026
+243 −133

Conversation

@dmaicher
Copy link
Contributor

@dmaicher dmaicher commented Jan 4, 2026 •
edited
Loading

Subject

Closes #7303

Changelog

### Changed
- the `symfony/security-acl` dependency is now optional. You need to explicitly require it as a dependency if you are using ACL.

Todo

VincentLanglet

This comment was marked as outdated.

Sign in to view

@dmaicher dmaicher mentioned this pull request Jan 5, 2026
Merged
@dmaicher dmaicher force-pushed the optional_acl branch 2 times, most recently from d60595d to d54f789 Compare January 5, 2026 15:40
VincentLanglet
VincentLanglet previously approved these changes Jan 5, 2026
View reviewed changes
@dmaicher dmaicher marked this pull request as ready for review January 5, 2026 15:51
@dmaicher dmaicher merged commit 1c288d9 into sonata-project:4.x Jan 5, 2026
24 checks passed
@dmaicher dmaicher deleted the optional_acl branch January 5, 2026 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@VincentLanglet VincentLanglet VincentLanglet approved these changes

Assignees

No one assigned

Labels

dependencies enhancement minor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

make symfony/security-acl optional dependency

2 participants

@dmaicher @VincentLanglet