Skip to content
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

Multipart mode submission of mutations using graphql-codegen document operations has not been converted #112

Closed
reaink opened this issue Apr 14, 2021 · 1 comment · Fixed by #113

Comments

@reaink
Copy link

reaink commented Apr 14, 2021

Multipart mode submission of mutations using graphql-codegen document operations has not been converted and submission is not supported

currently

operations: {"query":{"kind":"Document","definitions":[{"kind":"OperationDefinition","operation":"mutation","name":{"kind":"Name","value":"mergeSubmit"},"variableDefinitions":[{"kind":"VariableDefinition","variable":{"kind":"Variable","name":{"kind":"Name","value":"jobId"}},"type":{"kind":"NonNullType","type":{"kind":"NamedType","name":{"kind":"Name","value":"String"}}}},{"kind":"VariableDefinition","variable":{"kind":"Variable","name":{"kind":"Name","value":"xlsxFile"}},"type":{"kind":"NonNullType","type":{"kind":"NamedType","name":{"kind":"Name","value":"Upload"}}}}],"selectionSet":{"kind":"SelectionSet","selections":[{"kind":"Field","name":{"kind":"Name","value":"mergeSubmit"},"arguments":[{"kind":"Argument","name":{"kind":"Name","value":"jobId"},"value":{"kind":"Variable","name":{"kind":"Name","value":"jobId"}}},{"kind":"Argument","name":{"kind":"Name","value":"xlsxFile"},"value":{"kind":"Variable","name":{"kind":"Name","value":"xlsxFile"}}}]}]}}]},"variables":{"jobId":"987","xlsxFile":null}}
map: {"1":["variables.xlsxFile"]}
1: (binary)

expected

operations: {"query":"mutation mergeSubmit($jobId: String! $xlsxFile: Upload!)  {\n  mergeSubmit(jobId: $jobId xlsxFile: $xlsxFile)}","variables":{"jobId":"987","xlsxFile":null}}
map: {"1":["variables.xlsxFile"]}
1: (binary)
@logaretm
Copy link
Owner

Thanks for reporting this, I have fixed the issue and it is now tagged in rc.16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants