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

Provide a framework for overriding default config values, persist them across application restarts #75

Merged
merged 5 commits into from
Oct 4, 2023

Conversation

vivekuppal
Copy link
Owner

@vivekuppal vivekuppal commented Oct 4, 2023

Any config values changed in the UI will be persisted to override.yaml.
This has the following benefits

  1. Persist any changes done in the UI across application restarts
  2. All user changes can be isolated from application default values

Misc other changes in the PR

  • Optimize the Config, Global vars singleton object constructors. Skip processing in constructor once they are initialized.
  • Introduce a utilities file for common utility methods.
  • Add recursive dictionary merge capability to utilities file.

@vivekuppal vivekuppal merged commit 374f5fb into main Oct 4, 2023
2 checks passed
@vivekuppal vivekuppal deleted the vu-config-override branch October 4, 2023 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants