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.

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

Handling required as a list correctly #345

Closed
holsky opened this issue May 4, 2015 · 3 comments
Closed

Handling required as a list correctly #345

holsky opened this issue May 4, 2015 · 3 comments

Comments

@holsky
Copy link
Contributor

holsky commented May 4, 2015

Currently it seems like "required" fields are properly handled as per json schema v3. However using the "required" list does not work:

"required" : ["locality", "region"],

Since this is not on a node-level, but on a class level I am not sure where to tackle the problem. If you give me a hint I will gladly submit a pull request
@joelittlejohn
Copy link
Owner

Hi there. Support for the required rule specified as an array was added by #325, so this should be working for you in 0.4.11. If you aren't yet using this version, could you update and try again?

@joelittlejohn
Copy link
Owner

Feel free to reopen if you're still having problems with 0.4.11

@holsky
Copy link
Contributor Author

holsky commented May 6, 2015

no works like a charm :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@joelittlejohn @holsky