-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Serializer] Update custom_normalizer.rst #15150
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
base: 5.2
Are you sure you want to change the base?
Conversation
Added a live example from an app using a custom normalizer. More details: https://stackoverflow.com/q/66781361/10543130
If you're creating a REST api with a custom response, here's an example | ||
of what you would have in the `normalize` function: | ||
|
||
public function normalize($product, string $format = null, array $context = []) |
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.
What is different from the lines above in your example?
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.
I wanted to specifically mention REST api since I spent a lot of time searching keywords before I found the custom normalizer
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.
I think then that the example is not necessary since it seems to me identical to the previous one.
Maybe more a rewording to add this mentions.
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.
I see. I thought the documentation in this part wasn't rich enough in examples since I had to do some extra googling before knowing how to return specific attributes (see lines 62 to 74).
Might be just me since I mainly work with node environments, but feel free to make some edits.
More details on why I made these changes.
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.
I don't think this is the correct place for it. It would be easier to use #[Groups]
attribute on your Product
class rather than building a custom normalizer for it.
Added a live example from an app using a custom normalizer.
More details: https://stackoverflow.com/q/66781361/10543130