Skip to content

Conversation

@OsamaRab3
Copy link
Member

Hot-reload supported through volume mounting
All services run using docker compose up command
API is available at

http://localhost:3000

@OsamaRab3 OsamaRab3 requested a review from gazzaar November 22, 2025 11:14
@OsamaRab3 OsamaRab3 linked an issue Nov 22, 2025 that may be closed by this pull request
@@ -0,0 +1,62 @@
version: "3.9"
Copy link
Member

Choose a reason for hiding this comment

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

I think docker-compose now doesn't need this line, but it's ok


services:
app:
image: postgres:13
Copy link
Member

Choose a reason for hiding this comment

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

postgres 13? that's way too old bro, and also we need to use the alpine version for both postgres and redis.
I think we should use postgres17-alpine

image: postgres:13
container_name: multi_tenant_db
environment:
POSTGRES_USER: user
Copy link
Member

Choose a reason for hiding this comment

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

I think also here we could make use of env variables?

DATABASE_HOST: app
DATABASE_PORT: 5432
DATABASE_USER: user
DATABASE_PASSWORD: postgres8090
Copy link
Member

Choose a reason for hiding this comment

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

The same here for env variables

Copy link
Member

@gazzaar gazzaar left a comment

Choose a reason for hiding this comment

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

Please fix Postgres version and use alpine

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.

Add Dockerfile for Database Setup

2 participants