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.

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

Support for PHP 8.1 enums #2404

Closed
IonBazan opened this issue Jan 13, 2022 · 4 comments
Closed

Support for PHP 8.1 enums #2404

IonBazan opened this issue Jan 13, 2022 · 4 comments
Labels
Feature
Milestone
2.4.0

Comments

@IonBazan
Copy link
Member

Feature Request

Q A
New Feature yes
RFC yes
BC Break no

Summary

We could add native support for PHP 8.1+ enums by adding a new parameter to the mapping that allows to define the enum class name like it's done in ORM: doctrine/orm#9304
@ozahorulia
Copy link

I'm already using this feature in the latest doctrine ORM. It would be great to have the same functionality for the ODM.

@malarzm
Copy link
Member

malarzm commented Jan 29, 2022

Referencing ORM's PR we could base our work on: doctrine/orm#9304

@IonBazan IonBazan mentioned this issue Feb 20, 2022
Closed
@stale
Copy link

stale bot commented Apr 18, 2022

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

@stale stale bot added the Stale Issues that have not seen any activity in 30 days label Apr 18, 2022
@malarzm malarzm added this to the 2.4.0 milestone Apr 18, 2022
@stale stale bot removed the Stale Issues that have not seen any activity in 30 days label Apr 18, 2022
@malarzm
Copy link
Member

malarzm commented May 3, 2022

Merged in #2440

@malarzm malarzm closed this as completed May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature
Projects
None yet
Milestone
2.4.0
Development

No branches or pull requests
3 participants
@IonBazan @ozahorulia @malarzm