-
Notifications
You must be signed in to change notification settings - Fork 497
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
Supabase doc updates #1224
Supabase doc updates #1224
Conversation
Hey, here’s your docs preview: https://clerk.com/docs/pr/1224 |
…e all non-nextjs app router examples
Co-authored-by: Alexis Aguilar <[email protected]>
Co-authored-by: Alexis Aguilar <[email protected]>
Co-authored-by: Alexis Aguilar <[email protected]>
…rk/clerk-docs into sb-integration-guide-updates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really impressive work here 👏
It's quite an involved guide and I wonder if there's anything we can do to like, automate some of these steps or make it less of an episode to get the integration done, but after reading it through nothing is jumping out to me. It may just be what it is!
|
My mistake on closing this! We could build a small CLI that hooks into the database using the connection string to run the necessary SQL on the required tables. Do we have a way of determining how popular this integration is? Maybe checking how many people have a supabase jwt template? |
Important
🔎 Preview:
I walked through this document for the first time today and found a number of areas that could be expanded on or improved for clarity.
This includes: