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

Add structure to CellDetails #266

Open
MK-2001 opened this issue Jan 16, 2024 · 2 comments
Open

Add structure to CellDetails #266

MK-2001 opened this issue Jan 16, 2024 · 2 comments

Comments

@MK-2001
Copy link
Contributor

MK-2001 commented Jan 16, 2024

Describe the bug
Currently all Temps / Volts are within one folder in ioBroker.Objects are listed down. For my behavior it is uncomfortable to review these values and determine which module is affected.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'Objects'
  2. Click on 'bydhvs.x.CellDetails'
  3. Scroll down to 'your value'

Expected behavior
With the already determined information it should be possible to add an additional folder per module.

Versions:

  • Adapter version: 1.5.0

Additional context
If you @christianh17 as adapter owner agree I will create a PR with this solution in it.

@christianh17
Copy link
Owner

I am not sure, it breaks the scripts and logic if we change the structure of the objects. And: In my opinion the right way is not to use the objects "itself" but some kind of visualisation depending on the objects.
On the other hand: You did a great job in enhancing the adapter and I can see: Other people do use the objects itself.
What about a compromise: A switch in the configuration from the adapter and one can choose between the "old" (actual) behaviour and a more structured way for the cell details.

@christianh17
Copy link
Owner

The issue is some days older, there have been changes. Do you think a change is still neded?

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

No branches or pull requests

2 participants