Skip to content
This repository has been archived by the owner on Apr 14, 2023. It is now read-only.

apollo-link-error should provide more context to the error handler #127

Closed
glasser opened this issue Oct 5, 2017 · 4 comments
Closed

apollo-link-error should provide more context to the error handler #127

glasser opened this issue Oct 5, 2017 · 4 comments
Assignees
@jbaxleyiii
Milestone
Apollo Client 2.0...

Comments

@glasser
Copy link
Member

glasser commented Oct 5, 2017

The apollo-link-error onError ErrorHandler would be more useful if it received the operation, and maybe some more parts of the result if available.
@nevir
Copy link
Contributor

nevir commented Oct 9, 2017

👍

Our use case: We generate a UUID for each request, include it as a request-id header, but also need access to it when reporting errors

@stubailo
Copy link
Contributor

stubailo commented Oct 9, 2017

Yeah it should probably receive basically everything!

@jbaxleyiii
Copy link
Contributor

on it!

@jamesreggio jamesreggio mentioned this issue Oct 11, 2017
Closed
@jbaxleyiii jbaxleyiii added this to the Apollo Client 2.0 Launch milestone Oct 12, 2017
@jbaxleyiii
Copy link
Contributor

Fixed via #144 🎉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jbaxleyiii jbaxleyiii

Labels
None yet
Projects
None yet
Milestone
Apollo Client 2.0 Launch
Development

No branches or pull requests
4 participants
@glasser @nevir @stubailo @jbaxleyiii