-
-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
Settings creation #736
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
From #567
When a specific setting is requested via GET, if it isn't found, we check to make sure it isn't specified in the default settings JSON, if it is, we return the default value and load that into the DB
When trying to create settings via PUT/POST, if a setting is not yet in the DB, we should again check if it is specified in the default settings JSON. If it is we should create it, but with the supplied value, if it isn't we probably should not create it for now.
We can add advanced permissions later for creating new settings? Although it really shouldn't be necessary as apps will have their own place to create settings.
The text was updated successfully, but these errors were encountered: