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

[NRPTI-1115] DEV FEATURE** Create Dockerfiles and docker-compose #1116

Closed
wants to merge 2 commits into from

Conversation

LocalNewsTV
Copy link
Contributor

@LocalNewsTV LocalNewsTV commented Sep 27, 2023

Pull Request Standards

  • The title of the PR is accurate
  • The title includes the type of change [HOTFIX, FEATURE, etc]
  • The PR title includes the ticket number in format of [NRPTI-###]
  • Documentation is updated to reflect change

Description

This PR includes the following proposed change(s):

  • Creation of a Dockerfile, docker-compose.yaml, and create-db.js seed file.
  • The Dockerization will allow creation of a Mongodb database and its respective collections set by default, reducing the onboarding time of a developer in manually getting the docker image, and setting up their database for development.

Benefits

  1. The Database can be readily setup much faster than the current manual process
    • The developer still needs to seed their own data
  2. No need to worry about correctly installing dependencies in the API when doing development work in the front end.
  3. Docker images are grouped together appropriately

image

Note

  • The commented out section in the docker-compose is in hopes of the API later getting a health endpoint (to maintain containers)

@sonarcloud
Copy link

sonarcloud bot commented Oct 3, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 3 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@LocalNewsTV LocalNewsTV closed this Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants