-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[API] Split read serialization group #15770
Conversation
Bunnyshell Preview Environment deletedAvailable commands:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some entries in the test code, could be could to get rid of obsolete groups here as well:
<attribute name="groups">foo:read</attribute> |
Sylius/src/Sylius/Bundle/ApiBundle/Tests/Application/config/api_platform/FooSyliusResource.xml
Line 19 in 51a41dc
<attribute name="groups">foo-sylius-resource:read</attribute> |
And one entry in documentation:
Let's say that you want to change the serialized fields in your responses. |
Thank you, @TheMilek! |
…nd :show serialization groups (TheMilek) This PR was merged into the 1.13 branch. Discussion ---------- | Q | A | |-----------------|--------------------------------------------------------------| | Branch? | 1.13 <!-- see the comment below --> | | Bug fix? | no | | New feature? | no | | BC breaks? | no | | Deprecations? | yes <!-- don't forget to update the UPGRADE-*.md file --> | | License | MIT | <!-- - Bug fixes must be submitted against the 1.12 branch - Features and deprecations must be submitted against the 1.13 branch - Make sure that the correct base branch is set To be sure you are not breaking any Backward Compatibilities, check the documentation: https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html --> BC layer for #15770 Commits ------- Add BC layer for transition from :read to :index and :show serialization groups Add parameters that allow skipping adding read ord index and show serialization groups Add information to upgrade file
here is only split of the read group to show and index, in another PR there is a BC layer to handle it