ai-chatbot icon indicating copy to clipboard operation
ai-chatbot copied to clipboard

Add necessary DB migration step for localhost instructions

Open b8zhong opened this issue 11 months ago • 4 comments

Summary

This PR adds an explicit database migration step to the local setup instructions. This is necessary because without running migration - you will get DB errors without doing so.

Changes

Updates the "Running locally" section of the README to include the database migration step:

- pnpm install
- pnpm dev
+ pnpm install
+ pnpm tsx lib/db/migrate.ts && pnpm run build
+ pnpm dev

Why this is necessary

  1. On local deployment, the database schema needs to be initialized before authentication can work
  2. If not, you get authentication failures on first setup (#450, #731)

Testing Done

  • [x] Verified fresh setup works with these instructions
  • [x] Confirmed authentication works after following updated steps

Closes #450

b8zhong avatar Jan 28 '25 00:01 b8zhong

@b8zhong is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Jan 28 '25 00:01 vercel[bot]

@jeremyphilemon Sorry to bother you but noticed you were active here; would you mind taking a look? Thanks in advance!

b8zhong avatar Jan 28 '25 00:01 b8zhong

Thank you

adam-from-meiro avatar Jan 31 '25 12:01 adam-from-meiro

@adam-from-meiro Glad you found it helpful :)

b8zhong avatar Jan 31 '25 14:01 b8zhong