Skip to content

Add operation id in header#319

Closed
gridNA wants to merge 4 commits intoapollographql:masterfrom
gridNA:add-operationID-in-header
Closed

Add operation id in header#319
gridNA wants to merge 4 commits intoapollographql:masterfrom
gridNA:add-operationID-in-header

Conversation

@gridNA
Copy link
Copy Markdown

@gridNA gridNA commented Jul 17, 2018

/label feature

In our team we have a use case, when we need to send operationID in header with every query request. Because we can set up headers only during the ApolloClient init, we can't send operationID header dynamically. With my pr I've added operationID (if exists) in header for every request. The same logic is currently implemented in Android client.

@ghost ghost added the feature New addition or enhancement to existing solutions label Jul 17, 2018
@designatednerd
Copy link
Copy Markdown
Contributor

@gridNA does the work in #602 to allow header modifications cover this use case?

@designatednerd
Copy link
Copy Markdown
Contributor

This has been implemented in #658 (with SHOUTY HEADER NAMES to match Android) and shipped in 0.13.0. Gonna close this one out, thanks for your work!

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

Labels

feature New addition or enhancement to existing solutions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants