-
Notifications
You must be signed in to change notification settings - Fork 7
Add local setup to README #66
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
base: main
Are you sure you want to change the base?
Add local setup to README #66
Conversation
samholmes
left a comment
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.
Some initial review. I tried following the instructions to install supabase start and got stuck at this point:
❯ supabase start
WARNING: You are running different service versions locally than your linked project:
supabase/gotrue:v2.182.1 => v2.184.0
Run supabase link to update them.
[+] Pulling 14/17
✔ edgeRuntime Skipped - Image is already present locally 0.0s
✔ pgmeta Skipped - Image is already present locally 0.0s
✔ auth Skipped - Image is already present locally 0.0s
✔ storage Skipped - Image is already present locally 0.0s
✔ imgProxy Skipped - Image is already present locally 0.0s
✔ studio Skipped - Image is already present locally 0.0s
✔ api Skipped - Image is already present locally 0.0s
✔ realtime Skipped - Image is already present locally 0.0s
✔ kong Pulled 6.2s
✔ 9b18e9b68314 Pull complete 0.7s
✔ 7fd91e922960 Pull complete 0.7s
✔ 450997ae687c Pull complete 2.7s
✔ d8380bfcbd9b Pull complete 2.7s
⠙ mailpit Pulling 6.2s
✔ db Skipped - Image is already present locally 0.0s
⠙ logflare Pulling 6.2s
⠙ vector Pulling 6.2s
Starting database...
Initialising schema...
Seeding globals from roles.sql...
Applying migration 20250122000000_add_user_id_to_member.sql...
Stopping containers...
ERROR: relation "member" does not exist (SQLSTATE 42P01)
At statement: 0
-- Add user_id foreign key to member table (nullable initially)
ALTER TABLE member
ADD COLUMN user_id UUID REFERENCES auth.users(id) ON DELETE CASCADE
Try rerunning the command with --debug to troubleshoot the error.
Did you run into a similar issue?
I'm assuming I'm doing something wrong, so if you know what I did wrong please let me know.
I have Docker installed, and I ran brew install supabase/tap/supabase first then I ran supabase start which resulted in the failure you see above.
| - **Windows (npm)** – requires Node.js: | ||
|
|
||
| ```sh | ||
| npm install -g supabase | ||
| ``` |
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.
This isn't correct according to the docs: https://github.com/supabase/cli?tab=readme-ov-file#install-the-cli Perhaps just reference the docs to install supabase here. No need for use to copy the instructions in this README.
| `supabase/config.toml` is already configured to read these env vars via: | ||
| ```toml | ||
| [auth.external.github] | ||
| enabled = true | ||
| client_id = "env(SUPABASE_AUTH_EXTERNAL_GITHUB_CLIENT_ID)" | ||
| secret = "env(SUPABASE_AUTH_EXTERNAL_GITHUB_SECRET)" | ||
| ``` |
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.
I'm not sure this needs mentioning if it's already apart of the config. However, I double checked, and I don't see this in the toml file.
| supabase --version | ||
| ``` | ||
|
|
||
| #### 2. GitHub OAuth (local-only) |
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.
Username/password should be the recommended authentication for testing and should be mentioned here. We should put GitHub OAuth and Google OAuth setup in a separate docs/oauth-setup-guide.md file and link to it here.
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.
Ah separation of responsibilities (once again for me). I am in favor of having a separate docs folder for sub-setups (e.g. setting up GitHub OAuth, Google OAuth locally) in different folders. What we can do is if any contributor manages to find a way to do local setup for stuff like GitHub OAuth, then the contributor should add the exact steps to docs/oauth-setup-guide.md
Provided instructions on how to set up supabase environment for local development of DEVxSD.
Note
Adds comprehensive local development instructions for running Supabase alongside the app.
Docker Desktop,Supabase CLI) and verification (supabase --version)supabase/.envand referencingsupabase/config.tomlkeyssupabase start,supabase db reset) with expected service URLs.env.localand use the printedsb_publishable_...keyWritten by Cursor Bugbot for commit ea031d6. This will update automatically on new commits. Configure here.