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

[CSM] fix: Handle enum as values #1042

Merged
merged 3 commits into from May 17, 2022

Conversation

Geekimo
Copy link
Contributor

@Geekimo Geekimo commented Dec 28, 2021

This PR aims to handle enums as value.
It's purpose is to prevent the make;entity command to fail because of manual use of Enum in entity attributes.
It may help when Doctrine will support Enums natively.

Copy link
Collaborator

@jrushlow jrushlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Geekimo for the work on this. Could you rebase this please? Also github is flagging your commits as Unverified due to lack of a commit signature. (Github vigilante mode)

@jrushlow jrushlow added the Status: Needs Work Additional work is needed label Feb 22, 2022
@jrushlow jrushlow changed the title fix: Handle enum as values [CSM] fix: Handle enum as values Feb 23, 2022
@jrushlow jrushlow added the Bug Bug Fix label Mar 3, 2022
@jrushlow jrushlow force-pushed the feat/attributes-enum-as-values branch from da3cf3b to 5b28174 Compare May 17, 2022 13:01
jrushlow and others added 2 commits May 17, 2022 09:01
Co-authored-by: Ryan Weaver <weaverryan+github@gmail.com>
Copy link
Collaborator

@jrushlow jrushlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @Geekimo!

@jrushlow jrushlow added Status: Reviewed Has been reviewed by a maintainer and removed Status: Needs Work Additional work is needed labels May 17, 2022
@jrushlow jrushlow merged commit e09a55b into symfony:main May 17, 2022
@jrushlow jrushlow mentioned this pull request May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Bug Fix Status: Reviewed Has been reviewed by a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants