Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Serializer] Document the encoders#6684

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
weaverryan merged 1 commit intosymfony:2.7fromGuilhemN:ENCODERS
Sep 18, 2016
Merged

Conversation

@GuilhemN
Copy link
Contributor

QA
Doc fix?no
New docs?yes
Applies to2.7 -> master
Fixed tickets

Extracted from#6428

Using the Serializer component is really simple. You just need to set up
the:class:`Symfony\\Component\\Serializer\\Serializer` specifying
which Encoders andNormalizer are going to be available::
which Encoders andNormalizers are going to be available::
Copy link
Member

Choose a reason for hiding this comment

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

I would do the opposite and use "encoders" and "normalizers" here.

@ogizanagi
Copy link
Contributor

Should we document the context options used by each of the built-in encoders ?

@GuilhemN
Copy link
ContributorAuthor

@xabbuh Thanks for the comments, they should be fixed now.

@ogizanagi I won't have the time to do much more for now but of course it could be a great addition !

@xabbuh
Copy link
Member

👍

Status: Reviewed

@GuilhemN
Copy link
ContributorAuthor

@xabbuh i forgot your last comment, i'll move the custom encoders section to its own cookbook tonight.
Status: Needs work

@GuilhemN
Copy link
ContributorAuthor

I moved theCustom encoders section to its own cookbook.@xabbuh is it still ok for you ? :)

Status: Needs review

*:doc:`/cookbook/serializer/index`

* :doc:`/cookbook/serializer`
* :doc:`/cookbook/serializer/introduction`
Copy link
ContributorAuthor

@GuilhemNGuilhemNJul 18, 2016•
edited
Loading

Choose a reason for hiding this comment

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

I'm not sure about this name, if someone has a better idea...

@GuilhemNGuilhemNforce-pushed theENCODERS branch 2 times, most recently from5f18a6a toc144474CompareJuly 19, 2016 13:27
@weaverryanweaverryan merged commitdbdbf68 intosymfony:2.7Sep 18, 2016
weaverryan added a commit that referenced this pull requestSep 18, 2016
…ick, weaverryan)This PR was merged into the 2.7 branch.Discussion----------[Finishing][Serializer] Document the encodersFinishing#6684 - just opening the PR to review and run tests.Commits-------10b2fa3 Minor language tweaksf99663a removing cookbook entries73172c6 updating links33e5b76 Moving files into the new structure2c66c83 Merge branch 'ENCODERS' ofhttps://github.com/Ener-Getick/symfony-docs into Ener-Getick-ENCODERSdbdbf68 [Serializer] Document the encoders
@weaverryan
Copy link
Member

Merged via#6976. Thanks @Ener-Getick!

@GuilhemN
Copy link
ContributorAuthor

Thanks ! Sorry i didn't take the time to solve the conflicts...

@GuilhemNGuilhemN deleted the ENCODERS branchSeptember 19, 2016 05:04
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@GuilhemN@ogizanagi@xabbuh@weaverryan@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp