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 subschema property validation failures to context #2577

Merged

Conversation

nlundquist
Copy link

...during failure of any and all match modes.

Also considered adding additional context to one match mode, but the limit of what could be expressed without putting requirements on schema authors is likely just the index of matching schemas. e.g

{ message: 'value matched more that one schema', context: { matchedSchemaIndexes: [0, 3, 4] } }

@hueniverse hueniverse merged commit 1455ff5 into hapijs:master Dec 1, 2021
@hueniverse hueniverse self-assigned this Dec 1, 2021
@hueniverse hueniverse added the feature New functionality or improvement label Dec 1, 2021
@hueniverse hueniverse added this to the 17.5.0 milestone Dec 1, 2021
hueniverse added a commit that referenced this pull request Dec 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants