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

update collection.rst - add cascade_validation opt#3518

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

Closed
jillro wants to merge1 commit intosymfony:2.2fromjillro:addcascadevalidation

Conversation

@jillro
Copy link

QA
Doc fix?no
New docs?no
Applies toall

It makes sense to describe cascade_validation option in collection
documentation.Valid constraint does not suit all needs, especially
when we have to set validation_groups for embed object/forms, and so
cascade_validation should not be that "hidden".

It makes sense to describe cascade_validation option in collectiondocumentation. `Valid` constraint does not suit all needs, especiallywhen we have to set validation_groups for embed object/forms, and socascade_validation should not be that "hidden".
@wouterj
Copy link
Member

👍

This PR is related to issue#2360

@weaverryanweaverryan mentioned this pull requestFeb 4, 2014
weaverryan added a commit that referenced this pull requestFeb 4, 2014
This PR was merged into the 2.3 branch.Discussion----------Rebasing#3518This is just a rebase of#3518 against the 2.3 branch so that it can be merged cleanly.Commits-------561d201 update collection.rst - add cascade_validation opt
@weaverryan
Copy link
Member

This is a really great change - the exact types of small changes that make a lot of sense and make the docs really great. So thank you Guillaume!

I rebased this against 2.3 (since 2.2 has reached end of life and this patch didn't apply cleanly to 2.3) and opened (and merged) PR#3538.

Thanks!

@jillrojillro deleted the addcascadevalidation branchFebruary 4, 2014 08:46
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jillro@wouterj@weaverryan

[8]ページ先頭

©2009-2025 Movatter.jp