fix(app): replace graphql-upload to graphql-upload-minimal #18 #16

mercurius-js / mercurius-upload

graphql-upload implementation plugin for Fastify & mercurius

https://mercurius.dev/

MIT License

29 stars 12 forks source link

Closed qlaffont closed 1 year ago

qlaffont commented 1 year ago

Fixes #16 Fixes #18

qlaffont commented 1 year ago

@mcollina @PabloSzx :wave:

qlaffont commented 1 year ago

4 days since PR, If in 1 week no review and merge I will do a fork :/

mcollina commented 1 year ago

(I've been on vacation for the last two weeks)

I don't understand why this PR fixes the two linked issues.

What are the difference with graphql-upload-minimal?

qlaffont commented 1 year ago

My bad didn't know !

Bug for fix Mercurius was crashing according how graphql-upload handle upload
ESM This fork of graphql-upload have better performance and handle both case (CJS/ESM)