8000 [Serializer] Updated the list of loaded encoders by AndreiIgna · Pull Request #10369 · symfony/symfony-docs · GitHub
[go: up one dir, main page]
Skip to content

[Serializer] Updated the list of loaded encoders #10369

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[Serializer] Updated the list of loaded encoders #10369

wants to merge 1 commit into from

Conversation

AndreiIgna
Copy link

@javiereguiluz javiereguiluz requested a review from HeahDude October 8, 2018 07:22
HeahDude
HeahDude approved these changes Oct 9, 2018
View reviewed changes
Copy link
Contributor
@HeahDude HeahDude 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 very much @AndreiIgna for this nice catch! Those encoders were indeed added in Symfony 3.2.

@HeahDude HeahDude added this to the 3.4 milestone Oct 9, 2018
@javiereguiluz
Copy link
Member

Thanks Andrei.

javiereguiluz added a commit that referenced this pull request Oct 10, 2018
@javiereguiluz
minor #10369 [Serializer] Updated the list of loaded encoders (Andrei…
a7e6bf1 
…Igna)

This PR was submitted for the master branch but it was merged into the 3.4 branch instead (closes #10369).

Discussion
----------

[Serializer] Updated the list of loaded encoders

Framework bundle preloads encoders https://github.com/symfony/framework-bundle/blob/master/Resources/config/serializer.xml#L102
but not all were added in doc page https://symfony.com/doc/master/serializer.html#adding-normalizers-and-encoders

Commits
-------

f4cf54a [Serializer] Updated the list of loaded encoders
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HeahDude HeahDude HeahDude approved these changes

Assignees
No one assigned
Labels
Serializer Status: Reviewed
Projects
None yet
Milestone
3.4
Development

Successfully merging this pull request may close these issues.
4 participants
@AndreiIgna @javiereguiluz @HeahDude @carsonbot
0