Skip to content
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

chore: migration symfony 7 #2425

Closed
wants to merge 2 commits into from
Closed

chore: migration symfony 7 #2425

wants to merge 2 commits into from

Conversation

pauljosephkrogulec
Copy link

@pauljosephkrogulec pauljosephkrogulec commented Nov 21, 2023

allowing support of Symfony 7

I am targeting this branch, because is for allowing the possibility of use in symfony 7 project and don't create a BC-break

Changelog

Allowing support of Symfony 7 (@pauljosephkrogulec)

### Added
- Support of Symfony 7

### Changed

### Deprecated

### Removed

### Fixed

### Security

TODO

Need to merge this before merging this PR

Paul-Joseph added 2 commits November 23, 2023 09:16
@VincentLanglet
Copy link
Member

You need to add the same entry in the CI to check SF7 is installed correctly.

Also, we will need to add support for those lib first: https://github.com/sonata-project/SonataMediaBundle/blob/4.x/composer.json#L88-L91

@VincentLanglet
Copy link
Member

Also, we will need to add support for those lib first: 4.x/composer.json#L88-L91

I added support for SF7 on

  • SonataBlockBundle
  • SonataAdminBundle

For SonataDoctrineORMAdminBundle we need AuditBundle which need the DoctrineExtensions
For ClassificationBundle we need SonataDoctrineORMAdminBundle

@pauljosephkrogulec pauljosephkrogulec marked this pull request as draft November 30, 2023 08:02
@phansys
Copy link
Member

phansys commented Dec 7, 2023

Thank you for taking care of this @pauljosephkrogulec.
Please, let us know if you need help finishing this job.

@phansys
Copy link
Member

phansys commented Dec 14, 2023

Closing in favor of #2430.
Thank you!

@phansys phansys closed this Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants