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

Throw more detailed error messages when encountering Json Schema errors #1707

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hyzx86
Copy link

@hyzx86 hyzx86 commented Jun 24, 2024

fixes: #1429

@hyzx86
Copy link
Author

hyzx86 commented Jun 24, 2024

Hi @bencyoung-Fignum I'm trying to deal with this issue, but since it's been a while I'm not sure if it still exists.
Also it doesn't seem to be in the same position looking at your screenshot.
Can you provide more info on this? Or maybe you pull my PR and try it.

@bencyoung-Fignum
Copy link

I don't think that will help me as there's no identifying names on the one that doesn't get a value. The only interesting thing about it is IsAnyType is true. However it has no properties or anything else naming it

image

@RicoSuter
Copy link
Owner

RicoSuter commented Jul 17, 2024

Is this still needed? I'm fine merging this as it's not really breaking or anything...
Maybe it produces way too much text in the exception message.

Co-authored-by: Rico Suter <mail@rsuter.com>
@hyzx86
Copy link
Author

hyzx86 commented Jul 17, 2024

Maybe it produces way too much text in the exception message.

I think these texts are worthwhile. they can be somewhat easier for developers to locate the problem.
After all, it has happened exceptionally 😅

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.

Could not find the JSON path of a referenced schema
3 participants