-
Notifications
You must be signed in to change notification settings - Fork 182
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
Export directive to support batch processing. #99
base: master
Are you sure you want to change the base?
Conversation
Any thoughts on this? It is worth resolving the conflicts and getting it sync'd up with master? |
I would love to see this feature... I'm a bit new to graphql, admittedly, but was a little surprised that this isn't part of the core feature set. I did see some discussions about adding to apollo instead, but as I do think it's better suited in core... Just my anonymous $.02. |
Hello @boneill42, some progress in this feature?? |
Sorry, I haven't made any progress beyond what you see here. |
b1f26c1
to
a7ce75e
Compare
Similar to:
graphql/graphql-js#462
This PR is an initial stab at an export directive. It exposes
variable_values
in the context, and builds up a list of values for the variable during processing of the query.