-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Serializer] Add an @Ignore annotation #28744 #13701
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
[Serializer] Add an @Ignore annotation #28744 #13701
Conversation
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.
@vasilvestre thanks for contributing these docs. I left some comments about some minor things, but overall it looks OK to me 👍
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.
small code changes required after renaming MyObj
in favor of MyClass
Update serializer.rst Update components/serializer.rst Co-authored-by: Oskar Stark <[email protected]> Update components/serializer.rst Co-authored-by: Antoine Makdessi <[email protected]> Update components/serializer.rst Co-authored-by: Antoine Makdessi <[email protected]> Update components/serializer.rst Co-authored-by: Oskar Stark <[email protected]> Update components/serializer.rst Co-authored-by: Antoine Makdessi <[email protected]> Update serializer.rst
@javiereguiluz I think this should be good now ! |
Remove default value (false) from ignoring rules in XML and YML files as I didn't put them in PHP code.
Valentin, thanks a lot for your contribution ... and congrats on your very first contribution to Symfony Docs 🎉 |
Complete symfony/symfony#28744
Fix #13585