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

Wire up type aliases support in EntityValueResolver #1790

Merged

Conversation

NanoSector
Copy link
Contributor

@NanoSector NanoSector commented Apr 11, 2024

This is an addendum to PR symfony/symfony#54545 targeted at issue symfony/symfony#51765 in the Symfony Doctrine Bridge, which adds type alias support to EntityValueResolver.

This code injects the doctrine.orm.resolve_target_entities configuration into the EntityValueResolver class.

@NanoSector NanoSector requested a review from nicolas-grekas May 1, 2024 13:57
@ostrolucky ostrolucky added the Status: On Hold Most likely waiting for upstream resolution label May 5, 2024
@NanoSector NanoSector force-pushed the feature/type-aliases-support branch from f748096 to 18f1ca0 Compare August 12, 2024 20:41
@NanoSector NanoSector changed the base branch from 2.12.x to 2.13.x August 12, 2024 20:43
@NanoSector NanoSector force-pushed the feature/type-aliases-support branch from 18f1ca0 to 142a3df Compare August 12, 2024 20:47
@NanoSector
Copy link
Contributor Author

@ostrolucky Sorry to bother you, I am wondering if any action is required on my part, considering the PR still has 'on hold' status? Its Symfony counterpart is marked as ready.

@ostrolucky
Copy link
Member

It's onhold because upstream PR is not merged yet

@ostrolucky ostrolucky force-pushed the 2.13.x branch 3 times, most recently from abb5de0 to 2983fa7 Compare September 29, 2024 08:56
@ostrolucky ostrolucky force-pushed the 2.13.x branch 2 times, most recently from 01c7700 to 7bfb8e8 Compare October 21, 2024 19:57
This is an addendum to PR symfony/symfony#51765 in the Symfony Doctrine Bridge, which adds type alias support to EntityValueResolver.

This code injects the doctrine.orm.resolve_target_entities configuration into the EntityValueResolver class.
@NanoSector NanoSector force-pushed the feature/type-aliases-support branch from 142a3df to 27c31c3 Compare March 24, 2025 20:39
@NanoSector
Copy link
Contributor Author

@ostrolucky The upstream PR has been merged; is there anything left to do on my side?

@ostrolucky ostrolucky removed the Status: On Hold Most likely waiting for upstream resolution label Mar 24, 2025
@ostrolucky ostrolucky changed the base branch from 2.13.x to 2.15.x April 2, 2025 16:16
@ostrolucky ostrolucky added this to the 2.15.0 milestone Apr 2, 2025
@ostrolucky ostrolucky merged commit 6cd8bc2 into doctrine:2.15.x Apr 2, 2025
15 checks passed
@NanoSector NanoSector deleted the feature/type-aliases-support branch April 2, 2025 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants