Skip to content

Comments

[OpenWeatherMap] Add unitHint for ChannelType definitions with ItemType 'Number:Dimensionless'#19423

Merged
jlaur merged 2 commits intoopenhab:mainfrom
cweitkamp:feature-owm-unit-hint
Oct 3, 2025
Merged

[OpenWeatherMap] Add unitHint for ChannelType definitions with ItemType 'Number:Dimensionless'#19423
jlaur merged 2 commits intoopenhab:mainfrom
cweitkamp:feature-owm-unit-hint

Conversation

@cweitkamp
Copy link
Contributor

[OpenWeatherMap] Added unitHint for ChannelType definitions with ItemType 'Number:Dimensionless'

Signed-off-by: Christoph Weitkamp github@christophweitkamp.de

…nsionless'

Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
@jlaur jlaur changed the title [OpenWeatherMap] Added unitHint for ChannelType definitions with ItemType 'Number:Dimensionless' [OpenWeatherMap] Add unitHint for ChannelType definitions with ItemType 'Number:Dimensionless' Oct 2, 2025
@jlaur jlaur added the enhancement An enhancement or new feature for an existing add-on label Oct 2, 2025
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

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

LGTM

@jlaur jlaur merged commit dee2a7b into openhab:main Oct 3, 2025
2 checks passed
@jlaur jlaur added this to the 5.1 milestone Oct 3, 2025
@cweitkamp cweitkamp deleted the feature-owm-unit-hint branch October 3, 2025 09:46
DrRSatzteil pushed a commit to DrRSatzteil/openhab-addons that referenced this pull request Oct 11, 2025
…pe 'Number:Dimensionless' (openhab#19423)

* Added unitHint for ChannelType definitions with ItemType 'Number:Dimensionless'
* Remove min/max from ChannelType definitions

Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement An enhancement or new feature for an existing add-on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants