Skip to content

Enable Nuxt Bridge integration and add composables #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

Merged
merged 5 commits into from
Feb 18, 2022

Conversation

Strift
Copy link
Contributor

@Strift Strift commented Dec 18, 2021

Hello,

I love Supabase and the integration you built for Vue & Nuxt.

I saw that your other repository had an open issue for Nuxt 3 support and I wanted to try and help.

Since the Nuxt module is using this Vue plugin, I added some new composables in here.

Furthermore, to enable the Bridge integration, I needed the Vue 2 app to provide supabase, so I added that as well. I think the mixin is the Vue 2 way of doing this, but I'm not sure.

I will also open a PR on the nuxt-supabase repository (will edit when done). I think this will make more sense in the light of it.

I didn't add tests but did a lot of manual testing.
The docs still needs to be updated, I'll be more than happy to do it if the implementation is deemed satisfying. :)

PS: I'm pretty new to open-source and still learning TypeScript. So if there's anything that needs updating, just let me know!

Copy link
Collaborator

@sduduzog sduduzog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@scottrobertson I've tested with my existing projects, both for Vue 2x and Vue 3x and all is well.

@Strift Please modify the package.json version, possibly to version 2.2.4 so that github actions pick up this PR as a possible release

@@ -0,0 +1 @@
v14.17.3
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick question, what impact does this file have for anyone who does not have this version on node installed, given they have nvm anyways?

Copy link
Contributor Author

@Strift Strift Jan 22, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you have NVM installed, you still need to run nvm use to use the version in .nvmrc. Unless you have your shell configured to auto-run the command, but that would still happen behind the scenes.

Now, if you don't have the correct version installed, this is the error message you would get:

Found '/home/strift/Code/OpenSource/vue-supabase/.nvmrc' with version <v14.17.3>
N/A: version "v14.17.3 -> N/A" is not yet installed.

You need to run "nvm install v14.17.3" to install it before using it.

@Strift
Copy link
Contributor Author

Strift commented Jan 22, 2022

Hello @sduduzog, thanks for taking the time to review the PR!

Sorry I took a bit of time to get back to this, I've been sick the past week. 😅

I've updated the package.json version to 2.2.4 as requested. I also updated the URLs to the repository which were outdated.

Cheers,

@sduduzog sduduzog merged commit 7f30d1f into supabase-community:main Feb 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants