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

Already on GitHub?Sign in to your account

Merged
javiereguiluz merged 3 commits intosymfony:masterfromvasilvestre:document-ignore-annotation
Sep 3, 2020
Merged

[Serializer] Add an @Ignore annotation #28744#13701

javiereguiluz merged 3 commits intosymfony:masterfromvasilvestre:document-ignore-annotation
Sep 3, 2020

Conversation

vasilvestre
Copy link
Contributor

@vasilvestrevasilvestre commentedMay 24, 2020•
edited
Loading

Nyholm and artandor reacted with heart emoji
Copy link
Member

@javiereguiluzjaviereguiluz left a 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 👍

Copy link
Contributor

@noniagriconomienoniagriconomie left a 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 renamingMyObj in favor ofMyClass

Update serializer.rstUpdate components/serializer.rstCo-authored-by: Oskar Stark <oskarstark@googlemail.com>Update components/serializer.rstCo-authored-by: Antoine Makdessi <antoine.makdessi@agriconomie.com>Update components/serializer.rstCo-authored-by: Antoine Makdessi <antoine.makdessi@agriconomie.com>Update components/serializer.rstCo-authored-by: Oskar Stark <oskarstark@googlemail.com>Update components/serializer.rstCo-authored-by: Antoine Makdessi <antoine.makdessi@agriconomie.com>Update serializer.rst
@vasilvestre
Copy link
ContributorAuthor

@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.
@javiereguiluzjaviereguiluz added this to the5.2 milestoneSep 3, 2020
@javiereguiluzjaviereguiluz merged commit6d11c55 intosymfony:masterSep 3, 2020
@javiereguiluz
Copy link
Member

Valentin, thanks a lot for your contribution ... and congrats on your very first contribution to Symfony Docs 🎉

vasilvestre and Jibbarth reacted with hooray emojivasilvestre reacted with heart emoji

@vasilvestrevasilvestre deleted the document-ignore-annotation branchSeptember 4, 2020 09:50
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@jennevdmeerjennevdmeerjennevdmeer left review comments

@OskarStarkOskarStarkOskarStark left review comments

@noniagriconomienoniagriconomienoniagriconomie left review comments

@javiereguiluzjaviereguiluzjaviereguiluz approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
5.2
Development

Successfully merging this pull request may close these issues.

[Serializer] Add an @Ignore annotation
6 participants
@vasilvestre@javiereguiluz@jennevdmeer@OskarStark@noniagriconomie@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp