-
Notifications
You must be signed in to change notification settings - Fork 48
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
Support file upload #110
Support file upload #110
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @obukhov - thanks for submitting this (and the corresponding one on nautilus/graphql). A few small comments but we should be able to get this in quickly once those tests are ready
@AlecAivazis finished with tests can you please have a look? |
Hi @AlecAivazis! Sorry for pinging you, I saw you are busy. Any update on this one? |
No worries about pinging me. Thanks for the reminder :) I will pull this down today and test it. Things looks good from a code perspective so we should be able to get this in very quickly |
@@ -1,13 +1,18 @@ | |||
package gateway |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you so much for writing these tests!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you again for a very thorough contribution. Looking forward to the next one :)
Fixes the issue #103
It requires the following PR to be merged first: nautilus/graphql#13
Introduces compatibility with multipart/form-data requests for file uploads according to the following specification: