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

Improve handling of settings #92

Open
rwb27 opened this issue Dec 19, 2024 · 0 comments
Open

Improve handling of settings #92

rwb27 opened this issue Dec 19, 2024 · 0 comments
Labels
roadmap Plans for the future

Comments

@rwb27
Copy link
Collaborator

rwb27 commented Dec 19, 2024

Currently, settings are managed by a dictionary that's injected into a Thing when it attaches to a server. I think there are some improvements we could make.

  1. Introduce a setting descriptor. This would make it simple to have a Property that's persisted in the settings file.
  2. Allow settings to be specified in LabThings config files. This could either be as a default (i.e. set the setting in the file if it's not already set), an overwrite, or an instruction to lock the setting so it becomes read-only. I think there are a few use cases when that would be really helpful.
  3. Construct a settings model for more reliable serialisation/deserialisation to/from JSON.
@rwb27 rwb27 added the roadmap Plans for the future label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
roadmap Plans for the future
Projects
None yet
Development

No branches or pull requests

1 participant