Skip to content
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

Dockerfile added #68

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

PruthviGBhat
Copy link

this solves the issue #67
closes #67

@yashpandey06
Copy link

@PruthviGBhat As far as I know, we aren’t using Docker for deployments in this project. Given that, adding a Dockerfile specifically for the frontend might not make much sense. Could you clarify the need for it here?

@Roaster05
Copy link
Contributor

@PruthviGBhat the dockerfile which you have implemented isnt really helping as it just builds the extension files, try to come up which has more coverage and helps in our backend architecture as well

@ParagGhatage
Copy link

@PruthviGBhat the dockerfile which you have implemented isnt really helping as it just builds the extension files, try to come up which has more coverage and helps in our backend architecture as well

Hey @Roaster05 ,

I've implemented a Dockerfile setup along with a docker-compose.yml to manage both the backend and frontend, enabling the servers to run together seamlessly.

I can submit a PR with Docker documentation.
Let me know your thoughts!

@Roaster05
Copy link
Contributor

@PruthviGBhat the dockerfile which you have implemented isnt really helping as it just builds the extension files, try to come up which has more coverage and helps in our backend architecture as well

Hey @Roaster05 ,

I've implemented a Dockerfile setup along with a docker-compose.yml to manage both the backend and frontend, enabling the servers to run together seamlessly.

I can submit a PR with Docker documentation. Let me know your thoughts!

then submit a PR for the same

@Roaster05 Roaster05 added the question Further information is requested label Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Dockerfile for easier deployment and consistency
4 participants