Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi, I was trying to use unicorn form validation with a model that has the
unique_together
model's Meta option.Unicorn validation didn't validate this option. I saw your validation code and i noted two things:
unique_together
option or other types of validations that work on more than one fields of model error messages are indexed by Django with key called__all__
and with existing validation this field was never readI fixed this two problems with the code that i modified in this pull request. I tried to run your tests and they didn't pass, i saw that this happens because the tests didn't enter in old code but now the tests enter and they don't pass. I haven't changed the tests.
Probably you need to change the documentation and tests if you merge this pull request