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

Fix Docker support for the app #88

Open
wants to merge 2 commits into
base: 2025
Choose a base branch
from

Conversation

wmoussa-gc
Copy link

DO NOT MERGE
The deployment script actually looks for the tmp_Dockerfile and uses it as is. However, since we don't have access to the deployment process implemented by the cloud team, it will be difficult to fix this without risking production or staging.

@wmoussa-gc
Copy link
Author

@shawnthompson Just an FYI on how I managed to get the app running locally. As discussed, the cloud team is not using MongoDB native but a Azure variant under their Cosmos DB service.

@shawnthompson
Copy link
Member

@wmoussa-gc, what are your thoughts about, instead of renaming. Keeping the names as-is for the cloud hosting and using the typical file naming like the renaming you did to allow us to try to run docker locally for development? This way we are touching the files the cloud is dependent on and giving us the allowance to use docker locally instead of installing Mongo and node directly on our machines?

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.

2 participants