-
Notifications
You must be signed in to change notification settings - Fork 57
createBatchingNetworkInterface #3
Comments
At the moment, the |
@Poincare I'd like to take a stab at this, is there a contribution guide I can follow? |
@ramikhalaf Awesome! There isn't really a contribution guide but, here's basically the equivalent of one for this particular issue: This is where the network interface is currently implemented. What you probably want to do is replace this class with a method like Please let me know if the above makes sense - I can answer questions along the way :) |
I see #9 was merged. Any chance of releasing this soon, so I can test it out. |
Great work. I opened a PR against apollo-client for the last necessary step (passing through the 'id' property on the request object when batching). |
I seen this was now released and am trying to get this to work. const client = createApolloClient(createBatchingNetworkInterface({
uri: apiUrl,
opts: {
credentials: "same-origin",
headers: req.headers,
},
batchInterval: 20,
})); to work with However I try to wrap or replace it with I have latest |
Sorry, missed |
Was able to test this and it is working. Will close this now. |
Would this also work with createBatchingNetworkInterface or just createNetworkInterface?
The text was updated successfully, but these errors were encountered: