-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Can't upload assets #396
Comments
I think this is due to the graphql-upload version used by apollo-server-express. See this issue: apollographql/apollo-server#4039 The gist of it is that the current version of apollo server (which Vendure uses) uses an outdated version of the graphql-upload library, which is incompatible with node v13+. This PR looks to be approved so as soon as it makes it into a version of Apollo server then we can update our dependency. |
Excelent, so my other path if I did not want to wait (or they take too long ) would be to revert node to 12.x .. Thanks, I'll try this, do you want to leave this open so we validate once the PR is applied or would you prefer to put it on hold? |
I just downgraded to node v12.18.2 and it's working now. I will retest once de PR is in place and re-open this issue if it still fails. Thanks |
I guess they are altogether removing inbuilt Although some person is claiming a library upgrade is fixing their issue |
Re-opening this because it looks like Apollo will not fix this until Server v3, and we don't want people to keep running into this when we launch Vendure v1.0. |
Looks like the Apollo team have decided to get this solved in v2: apollographql/apollo-server#4865 Hopefully then we'll just need to upgrade to the latest Apollo Server v2.x to resolve this. |
The fix is now available in apollo-server-core v2.21.0, but unfortunately we can't directly ugrade to that right now because we it is a transitive dependency via |
Describe the bug
I created a default instance and when I try to upload assets it does not work, it gives the following error:
To Reproduce
Steps to reproduce the behavior:
If debug is enabled the following is printed:
Expected behavior
The asset is uploaded to the asset library.
Environment (please complete the following information):
Additional context
after testing I added debug by adding logger config to config file (src/vendure-config.ts)
The text was updated successfully, but these errors were encountered: