-
Notifications
You must be signed in to change notification settings - Fork 85
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
Expose type and error constructor for graphql response errors #312
Conversation
Exposes types and the error constructor for the error being thrown when a GraphQL response contains errors. This allows consumers to easily distinguish between unrelated errors vs. graphql related errors, and allows for type-guarded access of the response data / response headers.
can you check the remaining errors? Do the tests pass for you locally? |
@gr2m yeah, working on it. thx for the heads up. I'm guilty of not running the tests locally 😅🧐 |
you can also run |
Add test for instanceof
@gr2m Done. I had to step away for a little bit. Fixed lint, fixed coverage drop due to the new export. Should be good for another pass. thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great PR, thanks a lot 💐
🎉 This PR is included in version 4.7.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Exposes types and the error constructor for the error being
thrown when a GraphQL response contains errors. This allows
consumers to easily distinguish between unrelated errors vs.
graphql related errors, and allows for type-guarded access of
the response data / response headers.
Fixes #311