Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

add ability to configure mongo directly via dburl instead of separate… #281

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

Conversation

aochsner
Copy link
Contributor

@aochsner aochsner commented Sep 29, 2020

… settings

Affects: <hygieia-core-version-number>.


Allows the admin to specify a mongodb url instead of specifying each individual setting. This setting will override any others if it's not empty. https://docs.mongodb.com/manual/reference/connection-string/ for reference. This way you dont' have to expose every single option and just let them specify whatever they needs as URI

@trafico-bot trafico-bot bot added the 🔍 Ready for Review Pull Request is not reviewed yet label Sep 29, 2020
Copy link
Contributor

@rvema rvema left a comment

Choose a reason for hiding this comment

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

can you address the review comment?

@trafico-bot trafico-bot bot added ⚠️ Changes requested Pull Request needs changes before it can be reviewed again and removed 🔍 Ready for Review Pull Request is not reviewed yet labels Oct 5, 2020
@aochsner aochsner requested a review from rvema October 5, 2020 22:55
@trafico-bot trafico-bot bot added 🔍 Ready for Review Pull Request is not reviewed yet and removed ⚠️ Changes requested Pull Request needs changes before it can be reviewed again labels Oct 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🔍 Ready for Review Pull Request is not reviewed yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants