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

Add support for "required": false while using draft3 with strict mode #134

Merged
merged 1 commit into from
Oct 25, 2014

Conversation

bkirz
Copy link
Contributor

@bkirz bkirz commented Oct 13, 2014

Using strict mode currently only allows optional properties through "additionalProperties" and "patternProperties". In draft 3, one could also use "required": false to explicitly mark a property as optional; this change makes that possible within strict mode.

@pd
Copy link
Contributor

pd commented Oct 25, 2014

Looks good. In #139 I just advised someone to avoid strict mode because of issues like this, but if we do it for draft3, I'll add similar support for draft4.

pd added a commit that referenced this pull request Oct 25, 2014
Add support for `"required": false` while using draft3 with strict mode
@pd pd merged commit f0630aa into voxpupuli:master Oct 25, 2014
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.

2 participants