-
Notifications
You must be signed in to change notification settings - Fork 78
containerized the application #145
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
✅ Deploy Preview for paisable ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
@archa8 Please review the pr |
|
@archa8 any issues or improvements regarding the pr .? |
archa8
left a comment
There was a problem hiding this 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
|
@archa8 Yes you are right I will update the docker file. |
|
@archa8 I have added nginx.conf and updated the docker file. Please review the pr. |
|
@archa8 any issues or improvements regarding the pr .? |
Avdhesh-Varshney
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!

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
Dockerfilefor the backend service usingnode:16-alpine.Dockerfilefor the frontend:node:18-alpine.nginx:alpine.docker-compose.ymlto orchestrate backend, frontend, and database services.