search

djangoindia / djangoindia.org

Official website of djangoindia community.
MIT License
6 stars 8 forks source link

feat: docker dev container #30

Closed SureshPradhana closed 5 days ago

SureshPradhana commented 1 week ago

Description

fixes #5 Introduces Docker development environment setup for the project. Dockerfiles (Dockerfile.frontend.dev, Dockerfile.backend.dev) and Docker Compose (docker-compose.yml) configurations are added. Documentation updates are made in contributing.md for usage details. Resolves (https://github.com/djangoindia/djangoindia.org/issues/5). Note: Docker production setup is pending.

Changes Made

How to Test

  1. Build Docker images: docker-compose up --build
  2. Start both services:docker-compose up
  3. Start frontend service: docker-compose up frontend
  4. Start backend service:docker-compose up backend
  5. Access frontend: http://localhost:3000
  6. Access backend: http://localhost:8000

Notes

DevilsAutumn commented 1 week ago

Thankyou for this contribution @SureshPradhana ! I had some comments but before this I want #25 and #26 to get merged and have postgres setup as primary db. After all this is done, I will review this PR in depth. Till then you can keep updating this PR as above mentioned changes get merged.

DevilsAutumn commented 1 week ago

Hi @SureshPradhana , Can you configure the docker according to the changes in #37 ? I have done some refactoring, moved files here and there and changed default db to postgres. I am assuming setting up postgres on local will be a pain for other contributors so I want to merge docker changes and refactoring changes one after the other, this will allow them run the project using docker without need to download and setup postgres locally.

SureshPradhana commented 1 week ago

@DevilsAutumn Okay, I will work on it.

SureshPradhana commented 6 days ago

@DevilsAutumn, I will be attending a wedding and might not be available for a few days. Could you please handle this pull request in the meantime? I will resume contributing when I return. Also, I have included changes from pull request #37 for testing here. let me know if everything is okay; otherwise, how should this be handled?

DevilsAutumn commented 6 days ago

@SureshPradhana No worries! I will complete this PR. 👍