Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Serializer] Added a ConstraintViolationListNormalizer #9480

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.

Sign up for GitHub

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

Jump to bottom
Closed
javiereguiluz opened this issue Mar 23, 2018 · 7 comments
Closed

[Serializer] Added a ConstraintViolationListNormalizer #9480

javiereguiluz opened this issue Mar 23, 2018 · 7 comments
Labels
hasPR A Pull Request has already been submitted for this issue. help wanted Issues and PRs which are looking for volunteers to complete them. Serializer
Milestone
4.1

Comments

@javiereguiluz
Copy link
Member

symfony/symfony#22150

It normalizes ConstraintViolationListInterface instances according to RFC7807.
@javiereguiluz javiereguiluz added this to the 4.1 milestone Mar 23, 2018
@javiereguiluz javiereguiluz added the help wanted Issues and PRs which are looking for volunteers to complete them. label May 16, 2018
@javiereguiluz
Copy link
Member Author

We need a volunteer to contribute these docs. Someone expert in the Serializer component: @dunglas, @lyrixx, @Simperfit? Thanks!

@lyrixx
Copy link
Member

lyrixx commented May 22, 2018 •
edited
Loading

Should we really document it ? I mean, it's just about doing $this->serializer->serialize($errors);

@javiereguiluz
Copy link
Member Author

I think you are right. In the future we may add some brief mention somewhere if we think it's necessary. Thanks!

@dunglas
Copy link
Member

dunglas commented May 28, 2018 •
edited
Loading

We should at least add it to the list of available normalizers. People using directly the component have no way to discover this feature.

@javiereguiluz
Copy link
Member Author

Let's reopen then. But we'll need a volunteer to do this. Thanks!

@javiereguiluz javiereguiluz reopened this May 28, 2018
@javiereguiluz javiereguiluz mentioned this issue May 30, 2018
Closed
@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label May 30, 2018
javiereguiluz added a commit that referenced this issue May 31, 2018
@javiereguiluz
minor #9855 Mention the new ConstraintViolationListNormalizer (javier…
7a74476 
…eguiluz)

This PR was squashed before being merged into the 4.1 branch (closes #9855).

Discussion
----------

Mention the new ConstraintViolationListNormalizer

This fixes #9480.

Commits
-------

0d00129 Mention the new ConstraintViolationListNormalizer
@javiereguiluz
Copy link
Member Author

Fixed by #9855.

@dunglas
Copy link
Member

dunglas commented May 31, 2018

Thank you very much @javiereguiluz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hasPR A Pull Request has already been submitted for this issue. help wanted Issues and PRs which are looking for volunteers to complete them. Serializer
Projects
None yet
Milestone
4.1
Development

No branches or pull requests
3 participants
@dunglas @javiereguiluz @lyrixx