-
Notifications
You must be signed in to change notification settings - Fork 308
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
Remove file handling #500
Comments
Sad to see it's gone but understandable. It was me who originally added a file upload feature to graphql-request #175 . If someone needs it I'm still maintaining https://www.npmjs.com/package/awesome-graphql-client which has a file upload support and I'm not planning to remove it |
Well, this wasted half a day of my life trying to figure out what was going wrong since the previous versions of this library supported file uploads. Maybe put something like a warning message on the console to indicate that the file handling is removed? This would help poor souls such as myself who are trying to use previously working code on a new project and pulling all their hair out not knowing what is blowing up. :) |
@haZya PR welcome |
Perceived Problem
File handling adds a lot of complexity:
Ideas / Proposed Solution(s)
Remove the feature until it can be properly done. Imolementation, tests, docs.
The text was updated successfully, but these errors were encountered: