-
-
Notifications
You must be signed in to change notification settings - Fork 266
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 resolve 'encoding' #612
Comments
I'm seeing the same issue.
Everything is working as expected but it would be great to make it go away. |
Yes, am getting the same with the latest Nextjs 13.1.6. Everything works though. |
I am using node 18+ version, It does not need pnpm add -D encoding With this you can run. |
Any update on this one @thorwebdev ? |
@cdedreuille no update so far, but I'll take a look this week. |
As @kamto7 suggested, using npm: |
this is to remove warning from supabase/supabase-js#612
Goes to production ready next13 app dir and removes encoding warning (supabase/supabase-js#612)
@robbiecren07 Did you get around to trying it? I just freshly installed it locally and have it running locally, but I still seem to be getting this Error |
Yup I forgot to reply, I was able to deploy with no issues after running the above command in terminal. |
Might be good if someone opens a PR to add this Library as a Dependency. Unless someone else wants to do this, I'm probably gonna open one soon-ish |
whwn i use npm i -D encoding everything is working . |
It's amazing how at the same time the error is old but is so current |
Facing the same issue with next 13.4.3 Import trace for requested module:
./node_modules/node-fetch/lib/index.js
./node_modules/cross-fetch/dist/node-ponyfill.js
./node_modules/@supabase/storage-js/dist/main/lib/helpers.js
./node_modules/@supabase/storage-js/dist/main/packages/StorageFileApi.js
./node_modules/@supabase/storage-js/dist/main/StorageClient.js
./node_modules/@supabase/storage-js/dist/main/index.js
./lib/supabase-storage.js
./app/api/upload/route.ts |
same issue on next
|
Same issue, but for use with Firebase. On |
I am having the same error message with Nextjs semver 13.4.1. However, no issues deploying, and everything works as intended in production. What gives? |
Ran into this as well, looking forward to the fix. Thank folks! |
@thorwebdev @soedirgo Any update on this one? |
Still happening on Next 13.4.15 |
For me, I am using the same version (Next.js 13.4.15) and |
Using Next.JS 13.4.16 and can confirm that |
Hey all, we're still looking into a solution that doesn't require |
Hey, I still have this error on Next 13.4.19. I can confirm that adding encoding with |
i switched to pnpm, this solved it |
Part of fixing supabase/supabase-js#612
Looks like this will be resolved soon? @soedirgo
So I can test, once released. |
Hey all, can you check if the errors are still there after upgrading supabase-js to v2.33.2? p.s. Please also try to uninstall |
This seems to have fixed the issue in my case! |
Fixed it for me as well! (with encoding not installed) |
Confirmed |
This sorted out the issue! Appreciated! |
|
@MildTomato realtime-js imports |
Module not found: Can't resolve 'encoding' in '/Users/thorwebdev/Documents/code/supabase/quickstarts/supabase-next-13/node_modules/.pnpm/node-fetch@2.6.7/node_modules/node-fetch/lib'
This warning is printed when using supabase-js in nextjs 13 project: supabase/auth-helpers#340
It's related to an old version of node-fetch being used in cross-fetch: node-fetch/node-fetch#412 lquixada/cross-fetch#146
It should be safe to ignore as it's an optional dependency in node-fetch, but might be worthwhile to see if we can get rid of the warning.
The text was updated successfully, but these errors were encountered: