-
Notifications
You must be signed in to change notification settings - Fork 80
Open
Description
@eduardoarandah was saying on Gitter:
Settings manager should default from a regular config/settings file
Because adding settings to a production app is pain
No only settings values, also fields definitions.
Right now, code goes to repository and settings fields definitions don’t because they live in database.
Defaulting from regular config file also gives us the power of .env file, caching, etc
At first I didn’t agree, but now I think he may be onto something. It would be A LOT easier to define each Setting in PHP, not JSON in the db. Then Setting::get()
would get the value from the DB, if it exists, otherwise it would fall back to the default
in the configuration file.
boghy933
Metadata
Metadata
Assignees
Type
Projects
Status
No status