-
-
Notifications
You must be signed in to change notification settings - Fork 70
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
import npm module fail #78
Comments
I'm running into the same issue. Some of the |
I'm trying to import
It works fine locally. |
I am having the same problem. Everything works locally by running |
Mmmh, am I hallucinating or actually Supabase mentioned recently that they (at least for now) dropped the NPM support flag from Deno? You can see in this other issue of mine that (inside local-dev docker created by I think you'll have to focus on figuring out how to correctly fetch dependencies from |
Waiting for npm support -- for Firebase Cloud Messaging (Notifications) ... kinda a big missing thing. :/ |
@Pictor13 you mean this comment here from @laktek NPM support would be such a great thing -- it would eliminate a lot of problems with missing dependencies. And here the corresponding open issue on denoland |
+1 for |
+1 for adding Deno's native npm support. For now it seems #82 (comment) will work to use Edit: nope, I couldn't make it work =/ |
Meanwhile https://deno.com/blog/npm-on-deno-deploy. I thought that supabase edge runtime is based off deno deploy. Does it mean we can expect supabase to add support soon too? |
I agree with @Maxim-Filimonov, I think it's a major selling point to support npm modules (in Deno) and that's a major win for Supabase as well! Hopefully, we'll see this enabled soon! |
https://supabase.com/changelog. I can see here that they are using npm:@supabase.. to import { createClient } so I am guessing its in the work. Is there any timeline? I can continue developing locally for the time being, if its going to be supported soon. |
Yep, this is currently worked on. I can't provide an accurate an ETA for it as there are few unknowns at the moment. Will post an update when we have some progress. @Aadim1 unfortunately that example wouldn't work in Supabase edge functions yet :( |
@laktek Thanks for the quick reply. Happy to know its in active development. Cheers. |
Native npm support would be crucial for supabase edge functions - perhaps any news? @laktek |
With the recent announcement here, does edge-runtime already support npm modules? |
yes, importing npm modules with |
How would this work with a nextjs project? Haven't found anything in the documentation |
Bug report
Describe the bug
To Reproduce
Expected behavior
import npm module successful
Screenshots
System information
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: