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

Handle Result fields #65

Closed
obmarg opened this issue Jul 5, 2020 · 1 comment
Closed

Handle Result fields #65

obmarg opened this issue Jul 5, 2020 · 1 comment
Labels
✨ enhancement New feature or request

Comments

@obmarg
Copy link
Owner

obmarg commented Jul 5, 2020

GraphQL allows the lookup of any field to fail, regardless of whether that field is nullable. If the field is nullable then it will be set to null. If not, then the null will bubble up to the first nullable parent. Any associated errors will be stored in the root list of errors, ideally with a path that specifies where the error occurred.

Since the optional invariants in the schema hold true, then cynic should be able to parse the data. However, it might be nice to support Result field types that can somehow extract the error from the root errors and move it onto the actual field that errors.

@obmarg obmarg added the ✨ enhancement New feature or request label Jul 5, 2020
@obmarg
Copy link
Owner Author

obmarg commented Oct 20, 2021

This report is not true, closing

@obmarg obmarg closed this as completed Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant