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

Fix required property validation #103

Merged

Conversation

alexmmm
Copy link
Contributor

@alexmmm alexmmm commented May 5, 2014

Fixes issue #102, i.e. the regression introduced by 61059b2

@Sinewyk
Copy link

Sinewyk commented May 13, 2014

This is crucial and I currently need it ... if this could be merged as fast as possible it would be awesome 😃

@Reggino
Copy link

Reggino commented Jul 28, 2014

👍 Thanks!

@cmfcmf
Copy link

cmfcmf commented Aug 4, 2014

Same problem here, would be great to get this merged!

@craigh
Copy link

craigh commented Aug 4, 2014

please merge!

@craigh
Copy link

craigh commented Aug 13, 2014

ping @justinrainbow

@adamthehutt
Copy link

+1

Really important to merge this one. Thanks @justinrainbow

justinrainbow added a commit that referenced this pull request Aug 25, 2014
@justinrainbow justinrainbow merged commit 252e023 into jsonrainbow:master Aug 25, 2014
@justinrainbow
Copy link
Collaborator

Sorry for the long delay - merging this and other PRs for a new release tonight

hakre pushed a commit to hakre/json-schema that referenced this pull request Mar 26, 2015
…-validation

Fix required property validation
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.

7 participants