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

chore: update Supabase examples. #45074

Closed

Conversation

thorwebdev
Copy link
Contributor

@thorwebdev thorwebdev commented Jan 20, 2023

Removing the outdated example and replacing it with a list of useful Next.js + Supabase examples and resources. Does that work for y'all?

Documentation / Examples

  • Make sure the linting passes by running pnpm build && pnpm lint
  • The "examples guidelines" are followed from our contributing doc

@ijjk ijjk added the examples Issue/PR related to examples label Jan 20, 2023
@leerob
Copy link
Member

leerob commented Jan 20, 2023

Unfortunately, this would block folks wanting to do create-next-app -e with-supabase type cloning, for example. I would recommend instead keeping an example up to date here 🙏 In the future, we will be removing some of the empty directory README only "examples".

@thorwebdev
Copy link
Contributor Author

That's a good point, thanks @leerob

I've removed the old with-supabase-auth-realtime-db example and replaced it with an up to date with-supabase example. PTAL

Let me know if you're okay with the name change or if you'd prefer to stick with with-supabase-auth-realtime-db? E.g. would it affect this listing: https://vercel.com/templates/next.js/supabase-authentication ?

@balazsorban44
Copy link
Member

Closing in favor of #51335

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue/PR related to examples locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants