-
Notifications
You must be signed in to change notification settings - Fork 61
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
Add docs on adopting middleware #79
Comments
This project is not just middleware, it replaces the standard |
Okay. That's helpful, but I still think having a short transition section in the docs would be helpful. Even reading that StackOverflow answer, it's not clear to me how variables and query text ought to be handled. It may be that react-relay-network-modern does something sensible under the covers, but that's not really spelled out anywhere that I can find. |
|
I may just be really dense, but hopefully this feedback is helpful. I've been using Relay with a
fetchQuery
function as detailed in the Relay docs. Now I'd like to add an Authorization header and came across this project to do so. Given it's middleware, I'd expect to set it up to wrap myfetchQuery
function, but there's no indication in the docs how to do that. I came across an answer on StackOverflow that suggests usingurlMiddleware
in place offetchQuery
. If that's really the intended approach, I think the docs could be clearer in indicating that.The text was updated successfully, but these errors were encountered: