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

[ism8] Extend example sitemap #17738

Merged
merged 1 commit into from
Nov 17, 2024
Merged

Conversation

holgerfriedrich
Copy link
Member

  • Add things, items, and sitemap for heat pump and solar module.
  • Use UOM for all items.

This PR adds example code needed for heat pumps. Implementation of new channels was done in #17688.
Closes: #17682

* Add things, items, and sitemap for heat pump and solar module.
* Use UOM for all items.

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
@lsiepel
Copy link
Contributor

lsiepel commented Nov 11, 2024

The examples have German labels. I know they where already there, but if you are looking at it, would it be possible to adjust hem to English?

@holgerfriedrich
Copy link
Member Author

@lsiepel Fair point. Though, the documentation of ism8 is available only in German as well, and most of the item names and descriptions seem to be derived from the docs. When I start translating this, it will certainly sound wrong.... Not sure how to proceed on this.
Not sure how popular those devices are outside of Germany.

@lsiepel
Copy link
Contributor

lsiepel commented Nov 17, 2024

Wolf devices are being sold here in NL. As the rest is already in german it is consistent, so let's take this improvement as is.

Copy link
Contributor

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

@lsiepel lsiepel merged commit 562ce03 into openhab:main Nov 17, 2024
5 checks passed
@lsiepel lsiepel added this to the 4.3 milestone Nov 17, 2024
@holgerfriedrich holgerfriedrich deleted the pr-ism-example branch November 17, 2024 11:14
matchews pushed a commit to matchews/openhab-addons that referenced this pull request Dec 16, 2024
* Add things, items, and sitemap for heat pump and solar module.
* Use UOM for all items.

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
weymann pushed a commit to weymann/openhab-addons that referenced this pull request Dec 25, 2024
* Add things, items, and sitemap for heat pump and solar module.
* Use UOM for all items.

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
cipianpascu pushed a commit to cipianpascu/openhab-addons that referenced this pull request Jan 2, 2025
* Add things, items, and sitemap for heat pump and solar module.
* Use UOM for all items.

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: Ciprian Pascu <contact@ciprianpascu.ro>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ism8] Support for heat pump power channels
2 participants