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

✨ Prepare version 3 #22

Merged
merged 5 commits into from
Aug 1, 2024
Merged

✨ Prepare version 3 #22

merged 5 commits into from
Aug 1, 2024

Conversation

Nek-
Copy link
Member

@Nek- Nek- commented Jul 29, 2024

In previous versions of Doctrine, it was hard to act on entity load.
Since doctrine changed its internal behavior, we managed to improve the situation as well on the bundle.

The new PostLoadDispatcherInjecttionListener class allows us to remove a huge part of this bundle that was previously super-mega-custom.

This is awesome! But it comes with a huge refactoring, and this is not even the last commit toward the next major version of this package.

Plans:

  • Remove old code and use modern versions of Doctrine
  • Support Symfony 7
  • Clean configuration API no more relevant
  • Support Doctrine 3+

Nek- added 2 commits July 29, 2024 11:11
In previous versions of Doctrine, it was hard to act on entity load.
Since doctrine changed its internal behavior, we managed to improve the
situation as well on the bundle.

The new PostLoadDispatcherInjecttionListener class allows us to remove a
huge part of this bundle that was previously super-mega-custom.

This is awesome! But it comes with a huge refactoring, and this is not
even the last commit toward the next major version of this package.
@Nek- Nek- marked this pull request as draft July 29, 2024 09:56
Nek- added 2 commits July 29, 2024 17:33
This is more compatibility with Symfony 7 that removes Doctrine
subscribers!
@Nek- Nek- marked this pull request as ready for review July 29, 2024 15:35
@Nek- Nek- merged commit 9f03bca into master Aug 1, 2024
3 checks passed
@Nek- Nek- deleted the feature/prepare-version-3 branch August 1, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant