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 error paths of nested properties #52

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

bsonntag
Copy link
Contributor

@bsonntag bsonntag commented Sep 1, 2021

This PR fixes the validate function to return property paths in dot notation.

This was necessary because ajv changed these paths to use / instead of ..

@bsonntag bsonntag added the bug Something isn't working label Sep 1, 2021
@bsonntag bsonntag force-pushed the bugfix/nested-properties-error-paths branch from 5dee0c9 to 63fa028 Compare September 1, 2021 15:54
@bsonntag bsonntag assigned Joanata and unassigned catarinapeixoto Sep 1, 2021
@bsonntag bsonntag requested a review from Joanata September 1, 2021 15:55
@bsonntag bsonntag force-pushed the bugfix/nested-properties-error-paths branch 2 times, most recently from a9e61a7 to 20e9fe5 Compare September 1, 2021 16:10
@bsonntag bsonntag force-pushed the bugfix/nested-properties-error-paths branch from 20e9fe5 to de16851 Compare September 1, 2021 16:24
@catarinapeixoto catarinapeixoto merged commit 6e46eba into master Sep 1, 2021
@catarinapeixoto catarinapeixoto deleted the bugfix/nested-properties-error-paths branch September 1, 2021 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants