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

Upgrade sensors_to_show to its own data model attribute #1124

Open
nhoening opened this issue Jul 12, 2024 · 0 comments
Open

Upgrade sensors_to_show to its own data model attribute #1124

nhoening opened this issue Jul 12, 2024 · 0 comments

Comments

@nhoening
Copy link
Contributor

GenericAsset.attributes should be for dev or custom features, and sensors_to_show has become an important feature.

Editing it among all the others is also cumbersome.

So we should move it to its own place, GenericAsset.sensors_to_show.

Note that there is already a property for sensors_to_show on Genericasset. That should keep on working, its contents need refactoring.

Maybe we can discuss if the data migration code can also migrate this data from attributes. Probably we should do that with some Python scripting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant