8000 Updated the serializer/* articles to Symfony 4 by javiereguiluz · Pull Request #8687 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

Updated the serializer/* articles to Symfony 4 #8687

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 21, 2017

Conversation

javiereguiluz
Copy link
Member

No description provided.

the ``Resources/config/`` directory of a bundle;
* All ``*.yml`` and ``*.xml`` files in the ``Resources/config/serialization/``
* All ``*.yaml`` and ``*.xml`` files in the ``Resources/config/serialization/``
Copy link
Member

Choose a reason for hiding this comment

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

Here it would be necessary to mention the mapping paths introduced by the user, the default translations path and the overridden ones, in the order they are loaded.

serializer.rst < 8000 /span>
Symfony provides a serializer to serialize/deserialize to and from objects and
different formats (e.g. JSON or XML). Before using it, read the
:doc:`Serializer component docs </components/serializer>` to get familiar with
its philosophy and the normalizers and encoders terminology.
Copy link
Member

Choose a reason for hiding this comment

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

I don't like this. It's not something to fix now, but the user should be able to get all their basic info right here on this article. Probably more of the components docs need to move here.


$ composer require serializer

Then, enable the serializer in the framework config:
Copy link
Member

Choose a reason for hiding this comment

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

This is needed? I see that indeed it is! Shouldn't we fix this with a recipe? It won't block this PR, but we should look into that.

Copy link
Member

Choose a reason for hiding this comment

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

Wait, no, I th 8000 ink this is wrong. I think it auto-enables when the serializer is present.

@weaverryan weaverryan merged commit b3ef6c3 into symfony:master Nov 21, 2017
weaverryan added a commit that referenced this pull request Nov 21, 2017
…iluz)

This PR was squashed before being merged into the master branch (closes #8687).

Discussion
----------

Updated the serializer/* articles to Symfony 4

Commits
-------

b3ef6c3 Fixed a RST reference
57c539f Fixed a reference
453884f Updated the serializer/* articles to Symfony 4
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.

4 participants
0