Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
[Serializer] add documentation for serializer callbacks#3701
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
[Serializer] add documentation for serializer callbacks#3701
Uh oh!
There was an error while loading.Please reload this page.
Conversation
cordoval commentedMar 22, 2014
you may cook/grill me now@wouterj@weaverryan 👶 |
components/serializer.rst Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
remove theneeds
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
If you use the datetime field type [...]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Btw, I don't like mentioning something from the Form component docs into the Serializer component docs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
oh, wait. Now I see what you mean by "type fields". Just use "If a property of an object contains a DateTime instance or [...]"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
hmm, now I think more about it, we should just remove the complete DateTime thing from this section and only use it in the code example.
components/serializer.rst Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
with
weaverryan commentedApr 2, 2014
I like it, great work Luis! |
… (cordoval)This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes#3701).Discussion----------[Serializer] add documentation for serializer callbacksrelatedsymfony/symfony#10229| Q | A || --- | --- || Doc fix? | no || New docs? | yes || Applies to | 2.3+ || Fixed tickets | self 👶 || License | CC-ASA 3.0 Unported | Sent using [Gush](https://github.com/gushphp/gush)Commits-------b865b40 add comma madness and lowercasing Wefe2029 plug new revisioneabdbd0 add serializer set callback documentation
relatedsymfony/symfony#10229
Sent usingGush