Skip to content

Conversation

@usersina
Copy link

Added a live example from an app using a custom normalizer.
More details: https://stackoverflow.com/q/66781361/10543130

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 = [])
Copy link
Contributor

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?

Copy link
Author

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

Copy link
Contributor

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.

Copy link
Author

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.

Copy link
Contributor

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.

@carsonbot carsonbot changed the title Update custom_normalizer.rst [Serializer] Update custom_normalizer.rst Oct 9, 2022
@OskarStark
Copy link
Contributor

This issues is quite old and I agree with @ndench that this is not the right place.
I am closing this one. Thanks for proposing, feel free to resubmit your PR against 6.4 branch.

@OskarStark OskarStark closed this Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants