Skip to content
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

Possible or necessary modifications after the release of @supabase/ssr #301

Closed
MAXOUXAX opened this issue Nov 1, 2023 · 3 comments · Fixed by #357
Closed

Possible or necessary modifications after the release of @supabase/ssr #301

MAXOUXAX opened this issue Nov 1, 2023 · 3 comments · Fixed by #357
Labels
question Further information is requested

Comments

@MAXOUXAX
Copy link

MAXOUXAX commented Nov 1, 2023

I'd like this issue to be used as a forum for discussing necessary or possible modifications after the release of @supabase/ssr, whose guide is available here

Can/should changes be made to this module?
If so, which ones?
What are the implications?

@MAXOUXAX MAXOUXAX added the question Further information is requested label Nov 1, 2023
@felixgabler
Copy link
Contributor

I would also really like to know. In the future, auth helpers will probably be deprecated but there are many other useful parts in this Nuxt module so I guess one can just make it sit atop @supabase/ssr by replacing createClient with createServerClient from @supabase/ssr (and in supabase.client.ts maybe rather with createBrowserClient)?

@jfoucher
Copy link

I just submitted a PR that fixes an issue when using nuxt SSR with docker-compose. Other than that, everything seems to work fine. This project seems more or less abandoned, though... Maybe @larbish or @atinux can clarify?

@atinux
Copy link
Collaborator

atinux commented Dec 27, 2023

It is not abandoned, just that the maintainer also work on other stuff, I commented on your PR :)

From what I see, the module works well anyway, but it could be nice to update for the @supabase/ssr.

I believe it could be updated on:

PR is more than welcome, this is a free and open community project, I will do my best to review it ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants