Skip to content

[Serializer] Docs for CSV and YAML encoders #7040

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] Docs for CSV and YAML encoders #7040

wants to merge 1 commit into from

Conversation

dunglas
Copy link
Member

@dunglas dunglas commented Oct 6, 2016

@dunglas dunglas changed the title [Serializer] Docs for CSV and YAML encoders. [Serializer] Docs for CSV and YAML encoders Oct 6, 2016
@wouterj wouterj added this to the 3.2 milestone Nov 9, 2016
wouterj
wouterj reviewed Nov 9, 2016
View reviewed changes
components/serializer.rst
with the serializer enabled.

.. versionadded:: 3.2
The ``YamlEncoder`` and ``CsvEncoder`` encoders were added in Symfony 3.2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"were introduced in Symfony 3.2."

@wouterj
Copy link
Member

wouterj commented Nov 9, 2016

👍
status: reviewed

@xabbuh
Copy link
Member

xabbuh commented Dec 12, 2016

Thank you Kévin.

xabbuh added a commit that referenced this pull request Dec 12, 2016
@xabbuh
feature #7040 [Serializer] Docs for CSV and YAML encoders (dunglas)
864a1e1 
This PR was submitted for the master branch but it was merged into the 3.2 branch instead (closes #7040).

Discussion
----------

[Serializer] Docs for CSV and YAML encoders

Docs for symfony/symfony#19197 and symfony/symfony#19326.

Commits
-------

5687972 [Serializer] Docs for CSV and YAML encoders.
xabbuh added a commit that referenced this pull request Dec 12, 2016
@xabbuh
[#7040] fix the versionadded directive
fcd1e05
@xabbuh xabbuh closed this Dec 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wouterj wouterj wouterj left review comments

@mickaelandrieu mickaelandrieu mickaelandrieu approved these changes

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

Successfully merging this pull request may close these issues.
5 participants
@dunglas @wouterj @xabbuh @mickaelandrieu @carsonbot