Skip to content

[Serializer] Clarify use of CustomNormalizer in serializer documentation #18555

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

Closed
wants to merge 1 commit into from

Conversation

benito103e
Copy link
Contributor

See discussion symfony/symfony#50929

@javiereguiluz javiereguiluz added this to the 6.4 milestone Jan 22, 2025
@carsonbot carsonbot changed the title Clarify use of CustomNormalizer in serializer documentation [Serializer] Clarify use of CustomNormalizer in serializer documentation Jan 22, 2025
@javiereguiluz
Copy link
Member

Benjamin, we're sorry we didn't merge your PR on time. We stopped all contributions to the Serializer docs because we were revamping the docs about it.

Although the change that you proposed here is correct, we're closing this without merging because we already added this information in the doc revamp. See https://symfony.com/doc/6.4/serializer.html#built-in-normalizers

Thanks!

@benito103e
Copy link
Contributor Author

@javiereguiluz,
No problem ! I've just checked and the documentation is now more comprehensible than what I had suggested on this point.

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.

3 participants