Skip to content

Conversation

@MukeshAbhi
Copy link
Contributor

PR: Dockerize Frontend and Backend with Docker Compose #139

Description

This pull request adds Docker support to the project for easier setup and consistent development environments.


Changes

  • Added a Dockerfile for the backend service using node:16-alpine.
  • Added a multi-stage Dockerfile for the frontend:
    • Stage 1: Build React app using node:18-alpine.
    • Stage 2: Serve built app using nginx:alpine.
  • Added a docker-compose.yml to orchestrate backend, frontend, and database services.
  • To run the project using Docker Compose:
    docker compose up --build

@netlify
Copy link

netlify bot commented Oct 20, 2025

Deploy Preview for paisable ready!

Name Link
🔨 Latest commit b511710
🔍 Latest deploy log https://app.netlify.com/projects/paisable/deploys/68f77cac1c2c3200082452a4
😎 Deploy Preview https://deploy-preview-145--paisable.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 95
Accessibility: 100
Best Practices: 100
SEO: 91
PWA: 80
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@MukeshAbhi
Copy link
Contributor Author

@archa8 Please review the pr

@MukeshAbhi
Copy link
Contributor Author

@archa8 any issues or improvements regarding the pr .?

Copy link
Member

@archa8 archa8 left a comment

Choose a reason for hiding this comment

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

Hey @MukeshAbhi, thank you for opening the PR. I am not well-versed with Docker, but won't the Nginx configuration for the frontend cause problems? Like while hitting refresh on a page, there might be file not found problems. For that you might need to create a .conf file for Nginx. Let me know what you think

@MukeshAbhi
Copy link
Contributor Author

MukeshAbhi commented Oct 21, 2025

@archa8 Yes you are right I will update the docker file.

@MukeshAbhi
Copy link
Contributor Author

MukeshAbhi commented Oct 21, 2025

@archa8 I have added nginx.conf and updated the docker file. Please review the pr.

@MukeshAbhi
Copy link
Contributor Author

@archa8 any issues or improvements regarding the pr .?

@archa8 archa8 linked an issue Oct 22, 2025 that may be closed by this pull request
5 tasks
Copy link
Member

@Avdhesh-Varshney Avdhesh-Varshney left a comment

Choose a reason for hiding this comment

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

LGTM!

@archa8 archa8 merged commit 031e010 into Code-A2Z:master Oct 24, 2025
4 checks passed
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.

Containerize the Application using Docker

3 participants