DEVPROD-9618 rework sharing admin config #8596
Open
+640
−123
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DEVPROD-9618
Description
The current implementation will be hard to use going forward: in your staging's local database there are a couple of documents that override the shared config.
This PR changes this. It adds a, "Environment Overrides" section to the admin page
This way it's clear what's overridden, and you can override other things too if you need to.
As before, the shared config is a noop on prod and the main staging. A change from before is that if a config document is overridden, there will be an extra round of parsing for it (into a
bson.M
and back tobson.Raw
). The logic for getting the overridden configuration is a lot simpler now.Testing
I deployed this to my staging and it works.