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

Debug print query parse error #288

Closed
wants to merge 1 commit into from
Closed

Debug print query parse error #288

wants to merge 1 commit into from

Conversation

u9g
Copy link
Contributor

@u9g u9g commented May 17, 2023

Gives much more clarity to people why their query failed.

From:
image
To:
image

@obi1kenobi
Copy link
Owner

Thanks for flagging this! The underlying issue is that the parse error just hides the inner error's text instead of printing it. I've tweaked this in #289 so that the new error text here will read Found a type coercion with sibling fields. Please move those fields inside the type coercion.

In #289 I've also fixed a couple more error types that had the same problem, so you won't see "Query failed to parse" and no other info anymore. The error messages aren't perfect yet (I need to figure out how to include line number info in the error message) but they are certainly better now than they were before. Thanks for the help!

@obi1kenobi obi1kenobi closed this May 17, 2023
@u9g u9g deleted the patch-1 branch May 17, 2023 04:19
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.

2 participants