Skip to content

[Validator] Added array validation example to raw values #7805

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

Conversation

wouterj
Copy link
Member

@wouterj wouterj commented Apr 15, 2017

This example was removed from the component's README and it has some value imo.

Finishes #7161

@Koc
Copy link
Contributor

Koc commented Apr 15, 2017

fixes #5354

@Koc
Copy link
Contributor

Koc commented Apr 15, 2017

@wouterj can you also add example of validation lists (All constraint)? http://knplabs.com/en/blog/symfony-validators-standalone - starting from "Also, if you have an array of all of the same type – like an array of email addresses – you should check out the..."

@xabbuh
Copy link
Member

xabbuh commented Jul 21, 2017

Thank you @wouterj.

@xabbuh xabbuh merged commit 06f67b4 into 2.7 Jul 21, 2017
xabbuh added a commit that referenced this pull request Jul 21, 2017
…(CoolGoose, javiereguiluz, wouterj)

This PR was merged into the 2.7 branch.

Discussion
----------

[Validator] Added array validation example to raw values

This example was removed from the component's README and it has some value imo.

Finishes #7161

Commits
-------

06f67b4 Moved array validation to the Raw values sub-guide
64f5ad9 Fixed some syntax issues
c8b6b45 missing constraint example from the old readme
@xabbuh xabbuh deleted the CoolGoose-patch-1 branch July 21, 2017 13:53
@xabbuh
Copy link
Member

xabbuh commented Jul 21, 2017

@Koc Do you want to open a new PR with the additions you have in mind?

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