feat!: remove file uploads feature

[jasonkuhrt]

closes #500

[anulman]

Any chance you can release a new version with this fix in it, e.g. v5.2.1 or v5.3.0? I'm having a build fail due to graphql-request's dependency on extract-files:

Failed to resolve entry for package "extract-files". The package may have incorrect main/module/exports specified in its package.json: Missing "." specifier in "extract-files" package

[jasonkuhrt]

@anulman can you confirm that the preview release works for you?

[anulman]

Looks like it! Builds are succeeding; I'll report back if I see any issues. Tysm 🙏

[avisra]

this seems like a fairly big regression for users who were dependent on uploading files. how long was that feature in play for before it got removed? I assume if I am currently uploading files through graphql-request, that I should not upgrade the package for awhile?

[jasonkuhrt]

@avisra correct.

[brett-james-rocketlab]

Seems like a very big feature to take out of the package.

[avisra]

file upload is an inevitability of building apps today. removing a feature like this, without a workaround, doesn't seem kosher. graphql packages leave a large code footprint - and replacing its implementation with another graphql library (which supports file upload) is not a task that can be completed overnight.

fully aware this is an open source library, and you are free to do with it what you want. but removing a feature like this should really have gone through voting... The issue about removing file upload wasn't even live on github for a week.

[jasonkuhrt]

Nothing about what you have today has been broken. If it works for you stick with it. And when the feature returns, if it works for you, use it. PRs welcome guys. Thanks.