-
Notifications
You must be signed in to change notification settings - Fork 17
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
Nuxt 3 support #14
Comments
I believe this already works perfectly on nuxt3 but @sduduzog will know for sure. |
I believe the current version should work with Nuxt 2 bridge (based on vue2) but not Nuxt 3:
Also, it would be nice by starting to support the bridge, Nuxt module exports composable other than basic injected |
Ah yeah sorry, I was getting confused with the Vue3 support we added in https://github.com/supabase-community/vue-supabase which this library uses. Shouldn't be too much work to get it working, since this Nuxt wrapper is super simple. |
@scottrobertson @pi0 I"m seeing this now. Yes, the current version works with the nuxt-bridge (although no migration is yet done there), but is not yet ready for nuxt 3. I've taken some time to read through nuxt3-supabase and I love what he'd done with the project and I'll try as much as possible to also help where I can |
@sduduzog how do you want to handle this? Do you want to build it, or should @wobsoriano PR in some changes if they like? I am guessing we need to either carry on supporting both 2 and 3, or have a new major release that is 3 only. The latter may be the easier but not as nice for bug fixes etc for us going forward. |
Thanks for your quick responses! We are working on a simplified example at the moment for nuxt3+supabase. Will ping you when ready and probably follow up with a PR to convert it back into the module.
Agreed. Regarding versioning, same as other nuxt modules, it worth a major version to directly target Nuxt3. Nuxt2 users can be hopefully still use latest as long as using Nuxt bridge (that makes CompositionAPI and even small vue3 compatibility). |
Hello everyone 👋 I have opened two PRs to address this issue:
I would love to have your feedback on it, and know what are the next steps to hopefully get the PRs merged. 🤞 I'm pretty new to OSS so there's probably a couple things that are still off—just let me know! Cheers, |
Hi! Currently, we are listing
nuxt-supabase
as official Nuxt+Supabase integration. Thanks for your works @scottrobertson @sduduzogNuxt 3 beta is out for a while and @wobsoriano already started a nice Supabase module for Nuxt 3 (nuxt3-supabase)
I would like to see if possible, we collaborate together to make a new Major version of this module for Nuxt 3 and Nuxt 2 Bridge support in a single place (superbase-community) to ensure it keeps maintained in good hands 💚
@wobsoriano If you could make an initial PR, I and @atinux can help to make it stable.
The text was updated successfully, but these errors were encountered: