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

[freeboxos] Add unitHint on few channel types #17259

Merged
merged 4 commits into from
Aug 20, 2024

Conversation

lolodomo
Copy link
Contributor

Also remove an unused channel type "bandwidth".
Also avoid unneeded unit conversion when updating channel state.

Signed-off-by: Laurent Garnier [email protected]

@lolodomo lolodomo added the enhancement An enhancement or new feature for an existing add-on label Aug 14, 2024
@lolodomo lolodomo requested a review from clinique as a code owner August 14, 2024 19:00
Also remove an unused channel type "bandwidth".
Also avoid unneeded unit conversion when updating channel state.

Signed-off-by: Laurent Garnier <[email protected]>
@lolodomo lolodomo force-pushed the freeboxos_channel_uom branch from d949655 to 5712d5c Compare August 14, 2024 19:19
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.

Thanks, please see my question below.

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

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.

LGTM, Is there more to this conversation about the unit or can we merge this?

@jlaur
Copy link
Contributor

jlaur commented Aug 20, 2024

LGTM, Is there more to this conversation about the unit or can we merge this?

@clinique - I see you liked that message (which is also ambiguous 😁)... but to be sure, did you see this comment: #17259 (comment)?

@clinique
Copy link
Contributor

LGTM, Is there more to this conversation about the unit or can we merge this?

@clinique - I see you liked that message (which is also ambiguous 😁)... but to be sure, did you see this comment: #17259 (comment)?

Don't this that I like being ambiguous :-). At the moment I don't get it as being so !
You can proceed with merge from my point of view.

@lsiepel lsiepel merged commit 9ed05ad into openhab:main Aug 20, 2024
5 checks passed
@lsiepel lsiepel added this to the 4.3 milestone Aug 20, 2024
@lolodomo lolodomo deleted the freeboxos_channel_uom branch August 20, 2024 15:48
digitaldan pushed a commit to digitaldan/openhab-addons that referenced this pull request Aug 29, 2024
* [freeboxos] Add unitHint on few channel types

Also remove an unused channel type "bandwidth".
Also avoid unneeded unit conversion when updating channel state.

Signed-off-by: Laurent Garnier <[email protected]>
pgfeller pushed a commit to pgfeller/openhab-addons that referenced this pull request Sep 29, 2024
* [freeboxos] Add unitHint on few channel types

Also remove an unused channel type "bandwidth".
Also avoid unneeded unit conversion when updating channel state.

Signed-off-by: Laurent Garnier <[email protected]>
Signed-off-by: Patrik Gfeller <[email protected]>
joni1993 pushed a commit to joni1993/openhab-addons that referenced this pull request Oct 15, 2024
* [freeboxos] Add unitHint on few channel types

Also remove an unused channel type "bandwidth".
Also avoid unneeded unit conversion when updating channel state.

Signed-off-by: Laurent Garnier <[email protected]>
matchews pushed a commit to matchews/openhab-addons that referenced this pull request Oct 18, 2024
* [freeboxos] Add unitHint on few channel types

Also remove an unused channel type "bandwidth".
Also avoid unneeded unit conversion when updating channel state.

Signed-off-by: Laurent Garnier <[email protected]>
cipianpascu pushed a commit to cipianpascu/openhab-addons that referenced this pull request Jan 2, 2025
* [freeboxos] Add unitHint on few channel types

Also remove an unused channel type "bandwidth".
Also avoid unneeded unit conversion when updating channel state.

Signed-off-by: Laurent Garnier <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
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.

4 participants