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

Category entity in constructor of CategoryRepository is resolved via EntityNameResolver #341

Merged
merged 1 commit into from Jul 27, 2018

Conversation

Miroslav-Stopka
Copy link
Contributor

@Miroslav-Stopka Miroslav-Stopka commented Jul 23, 2018

  • Category entity is now extendable and extraction of class metadata are loaded from an extended entity if exists
Q A
Description, reason for the PR Class metadata for GedmoTree are not available in extended Category entity
New feature No
BC breaks No
Fixes issues ...
Standards and tests pass Yes
Have you read and signed our License Agreement for contributions? Yes

@TomasLudvik TomasLudvik self-requested a review July 26, 2018 14:16
@TomasLudvik
Copy link
Member

Hello @Miroslav-Stopka,

thank you for your PR. This is a nice job I like it a lot.

Please update your CM slightly and I will be able to accept this.

Category entity in the constructor of CategoryRepository is resolved via EntityNameResolver

  • Category entity is now extendable and extraction of class metadata are loaded from an extended entity if exists

@Miroslav-Stopka
Copy link
Contributor Author

Done :) Can you confirm? Thank you.

@TomasLudvik TomasLudvik changed the title Category entity in constructor of CategoryRepository is resolved via … Category entity in constructor of CategoryRepository is resolved via EntityNameResolver Jul 27, 2018
Copy link
Member

@TomasLudvik TomasLudvik 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 for updating. It is ok.

@TomasLudvik
Copy link
Member

TomasLudvik commented Jul 27, 2018

I have tested it and it is even working well, good job there @Miroslav-Stopka!

Copy link
Contributor

@LukasHeinz LukasHeinz left a comment

Choose a reason for hiding this comment

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

What about changelog entry?

…via EntityNameResolver

- Category entity is now extendable and extraction of class metadata are loaded from an extended entity if exists
@Miroslav-Stopka Miroslav-Stopka merged commit 99d7d26 into master Jul 27, 2018
@Miroslav-Stopka Miroslav-Stopka deleted the msa-category-extendible branch July 27, 2018 15:05
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.

None yet

3 participants