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

Per validation mode required and recommended rules #335

Merged

Conversation

henryaddison
Copy link
Contributor

Update rules on requiredFields, recommendedFields and requiredOptions to use the definition specific to the current validationMode if there is one. Otherwise falls back on the default, top-level definition.

Part of openactive/data-models#41

…e is one present

otherwise just use the default one attached to model spec at root level
… if there is one present

otherwise just use the default one attached to model spec at root level
…here is one present

otherwise just use the default one attached to model spec at root level
@henryaddison
Copy link
Contributor Author

Hi @nickevansuk this PR provides updates to existing rules as discussed about openactive/data-models#41.

Rule for shallNotInclude to follow (left it on a different computer, doh!).

@henryaddison
Copy link
Contributor Author

Related to openactive/data-models#45

@henryaddison henryaddison merged commit fe08fcf into master Oct 25, 2019
@henryaddison henryaddison deleted the per-validation-mode-required-and-recommended-rules branch October 25, 2019 10:20
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.

3 participants