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.

Sign up for GitHub

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

Jump to bottom

[dsmr] Just tested it in 4.2.0-SNAPSHOT and the "Actual Power Delivery" channel is shown twice in the UI. #16731

Closed
lsiepel opened this issue May 9, 2024 · 2 comments · Fixed by #16739
Closed

[dsmr] Just tested it in 4.2.0-SNAPSHOT and the "Actual Power Delivery" channel is shown twice in the UI. #16731

lsiepel opened this issue May 9, 2024 · 2 comments · Fixed by #16739
Labels
bug An unexpected problem or unintended behavior of an add-on

Comments

@lsiepel
Copy link
Contributor

lsiepel commented May 9, 2024 •
edited 

          Just tested it in 4.2.0-SNAPSHOT and the "Actual Power Delivery" channel is shown twice in the UI.

image

Originally posted by @dequeckerp in #15038 (comment)

It seems the label is wrong here, the channel id is emeter_actual_demand @coop-git is that a good conclusion and are you able to come up with a PR if that is the case?
@lsiepel lsiepel added the bug An unexpected problem or unintended behavior of an add-on label May 9, 2024
@coop-git
Copy link
Contributor

coop-git commented May 9, 2024

Yes, it was a copy of another/existing channel type. I can come up with a PR but need to setup my environment to 4.2
Depending on personal/work planning this can take a while.

@coop-git coop-git changed the title Just tested it in 4.2.0-SNAPSHOT and the "Actual Power Delivery" channel is shown twice in the UI. [dsmr] Just tested it in 4.2.0-SNAPSHOT and the "Actual Power Delivery" channel is shown twice in the UI. May 9, 2024
@lsiepel lsiepel mentioned this issue May 10, 2024
Merged
@lsiepel
Copy link
Contributor Author

lsiepel commented May 10, 2024

Yes, it was a copy of another/existing channel type. I can come up with a PR but need to setup my environment to 4.2 Depending on personal/work planning this can take a while.

Just created a PR, maybe you can just have a look and confirm this is all it takes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug An unexpected problem or unintended behavior of an add-on
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[dsmr] Fix delivery demand labels lsiepel/openhab-addons
2 participants
@lsiepel @coop-git