Skip to content

Integrations#1

Open
dakdevs wants to merge 9 commits into
productionfrom
integrations
Open

Integrations#1
dakdevs wants to merge 9 commits into
productionfrom
integrations

Conversation

@dakdevs
Copy link
Copy Markdown
Owner

@dakdevs dakdevs commented Sep 10, 2025

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
better-profile-app Error Error Sep 10, 2025 5:32am

Copy link
Copy Markdown
Owner Author

@dakdevs dakdevs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the file path from the top of every file in the repo so that it is possible to clearly review this and get this merged.

You are going to need to fully reset the folder src/db/migrations back to what production is set to, then run pnpm db:migrate AFTER.

Comment thread docker-compose.yml Outdated
command: '-d 1'
ports:
- '5432:5432'
- '5434:5432'
Copy link
Copy Markdown
Owner Author

@dakdevs dakdevs Sep 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- '5434:5432'
- '${POSTGRES_PORT:-5432}:5432'

This will allow you to set your own port in .env and keep it defaulted to 5432

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move this back to src/db/migrations/.gitkeep

Comment thread .env.example
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't delete this file's contents.

Copy link
Copy Markdown
Owner Author

@dakdevs dakdevs Sep 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file should not be changed.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Reset the migrations folder to production's.
  2. pnpm db:push
  3. pnpm db:migrate

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file should not be replaced.

Comment thread src/db/models/index.ts
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: It would be nice if only the additions were added to this file instead of them all being rearranged, would make it clearer to read.

Comment thread docker-compose.yml Outdated
services:
postgres:
image: postgres
image: pgvector/pgvector:pg16
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
image: pgvector/pgvector:pg16
image: pgvector/pgvector

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants