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

[OmniLink] Fix OmniLink temperature/humidity sensors #10130

Merged
merged 1 commit into from
Feb 16, 2021

Conversation

ecdye
Copy link
Contributor

@ecdye ecdye commented Feb 12, 2021

The upstream code the allows connection to the OmniPro uses float values to convert into Omni values, therefore this change was necessary to allow for proper execution of setting temperature commands.

Signed-off-by: Ethan Dye mrtops03@gmail.com

Signed-off-by: Ethan Dye <mrtops03@gmail.com>
@ecdye ecdye added the external bug A problem or unintended behavior of an external library label Feb 12, 2021
@ecdye ecdye requested review from J-N-K and fwolter February 12, 2021 16:54
@ecdye ecdye added the rebuild Triggers Jenkins PR build label Feb 12, 2021
@Skinah Skinah removed the rebuild Triggers Jenkins PR build label Feb 12, 2021
@Skinah
Copy link
Contributor

Skinah commented Feb 12, 2021

Can you edit your first post to give why you are doing the fix? what symptoms? why is it needed? Someone may do a review that may not have the hardware and they will need some context and also it goes into the release notes that all users may click on to read the details.

@ecdye
Copy link
Contributor Author

ecdye commented Feb 15, 2021

@J-N-K @fwolter Any chance for a review?

@J-N-K
Copy link
Member

J-N-K commented Feb 16, 2021

I‘m no longer part of the maintainer group, so: no.

@ecdye ecdye requested review from cpmeister and removed request for J-N-K February 16, 2021 18:21
@ecdye
Copy link
Contributor Author

ecdye commented Feb 16, 2021

I‘m no longer part of the maintainer group, so: no.

I'm sorry that it came to this. Thanks for all your contributions to this project over the past couple of years. Wishing you the best for the future!

Copy link
Contributor

@cpmeister cpmeister left a comment

Choose a reason for hiding this comment

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

LGTM

@cpmeister cpmeister merged commit 6f9d80e into openhab:main Feb 16, 2021
@cpmeister cpmeister added this to the 3.1 milestone Feb 16, 2021
K4ntir4n pushed a commit to K4ntir4n/openhab-addons that referenced this pull request Feb 17, 2021
themillhousegroup pushed a commit to themillhousegroup/openhab2-addons that referenced this pull request May 10, 2021
Signed-off-by: Ethan Dye <mrtops03@gmail.com>
Signed-off-by: John Marshall <john.marshall.au@gmail.com>
thinkingstone pushed a commit to thinkingstone/openhab-addons that referenced this pull request Nov 7, 2021
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
Signed-off-by: Ethan Dye <mrtops03@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external bug A problem or unintended behavior of an external library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants