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

feat: added docker-compose environment for quick deployments #2320

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

Conversation

Vorstenbosch
Copy link

Hey!

I wanted to see what you have been building but I struggled with setting up a quick development environment. The install guide seems a bit outdated and I did not want to run all the dependencies on my own box.

To cater for my needs I created a docker compose environment to spin up the app (including a database) with a single command. I am not sure if you are interested but it might help other people to try your project out.

I have not spend too much time on this so if you are interested but you require some changes feel free to comment or make changes.

@buddhika75
Copy link
Member

Dear @Vorstenbosch,

Thank you for your valuable input and effort in enhancing the accessibility of our Health Information Management System (HIMS). We appreciate your initiative in creating a Docker compose environment to streamline the setup process. This is indeed a thoughtful addition, as it simplifies the initial setup for new users and potentially encourages more engagement with the project.

We are certainly interested in incorporating your contribution. To proceed, we will review the pull request to ensure compatibility and adherence to our project's standards. Should any modifications be necessary, we will provide feedback or make the required adjustments.

Your contribution aligns well with our goal of making the HIMS more user-friendly and accessible. We are committed to continuous improvement and rely on valuable community contributions like yours to achieve this.

Once again, thank you for your effort and for supporting the HIMS project. We look forward to merging your pull request after a thorough review.

@buddhika75
Copy link
Member

@Senula88 / @PasinduW99 / @DamithDeshan / @Thiwanka570 - can you please check and discuss with the QA team and update.

Copy link
Member

Choose a reason for hiding this comment

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

Checking on this

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.

None yet

2 participants