Skip to content
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

Rebasing #3518 #3538

Merged
merged 1 commit into from
Feb 4, 2014
Merged

Rebasing #3518 #3538

merged 1 commit into from
Feb 4, 2014

Conversation

weaverryan
Copy link
Member

This is just a rebase of #3518 against the 2.3 branch so that it can be merged cleanly.

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".
weaverryan added a commit that referenced this pull request Feb 4, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

Rebasing #3518

This 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 weaverryan merged commit 561d201 into 2.3 Feb 4, 2014
@weaverryan weaverryan deleted the guilro-addcascadevalidation branch February 4, 2014 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant