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

If setting not found use the config, if config not found throw exception #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yormy
Copy link

@yormy yormy commented Apr 12, 2019

Allow getting of the config(key) when the settings(key) is not yet defined. Makes the app more transparant and all config() can be replaced by settings(). The code decides if the key can be overridden by the database, otherwise it just gets the config().
Also allow to throw an exception if the config() is not defined. This reduces typos in development

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.

1 participant