Skip to content

Documented the framework.serializer.mapping config option #8529

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

Merged
merged 3 commits into from
Nov 16, 2017

Conversation

javiereguiluz
Copy link
Member

This fixes #7701.

@@ -1692,6 +1694,19 @@ method.
For more information, see
:ref:`component-serializer-handling-circular-references`.

mapping

Choose a reason for hiding this comment

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

The target mapping is duplicated (already used for the validation section)

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks! I've fixed it.

**type**: ``array`` **default**: ``[]``

This option allows to define an array of paths with files or directories where
the component will look for additional validation files.
Copy link
Member

Choose a reason for hiding this comment

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

serialization :)

@weaverryan weaverryan added this to the 3.3 milestone Nov 16, 2017
@weaverryan
Copy link
Member

Typo fixed! Thanks!

@weaverryan weaverryan merged commit 8a1b7e0 into symfony:3.3 Nov 16, 2017
weaverryan added a commit that referenced this pull request Nov 16, 2017
…on (javiereguiluz, weaverryan)

This PR was merged into the 3.3 branch.

Discussion
----------

Documented the framework.serializer.mapping config option

This fixes #7701.

Commits
-------

8a1b7e0 fix typo
d3f9522 Fixed the reference to a repeated config option
d5b6154 Documented the framework.serializer.mapping config option
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants