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

[shelly] Various small fixes for BLU HT, Range Extender, Plus 10V, TRV set temp #16746

Merged
merged 4 commits into from
May 28, 2024

Conversation

markus7017
Copy link
Contributor

@markus7017 markus7017 commented May 11, 2024

This PR includes various small fixes (1 liners)

Closing
#16736
#16734
#16542
#16709
#16655
#16724

…enhab#16735

Signed-off-by: Markus Michels <markus7017@gmail.com>
Signed-off-by: Markus Michels <markus7017@gmail.com>
…e with

Shelly Plus 2PM

Signed-off-by: Markus Michels <markus7017@gmail.com>
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, let's wait for some test confirmations.

@lolodomo
Copy link
Contributor

@markus7017 : What is the status here ? Still work in progress ? Or still waiting for additional tests ? Or just ready for a merge ?

Signed-off-by: Markus Michels <markus7017@gmail.com>
@markus7017
Copy link
Contributor Author

@lolodomo waited for tester feedback, looks good = ready to merge

@lolodomo lolodomo removed the work in progress A PR that is not yet ready to be merged label May 28, 2024
@lolodomo
Copy link
Contributor

@lsiepel : I let you click on merge button as you reviewed this PR.

You will probably have then to close each referenced issue manually as I believe this was not declared as expected for an automatic closing.

@markus7017
Copy link
Contributor Author

@lolodomo I marked the issues in the initial description (Closing:) or is the syntax incorrect?

@markus7017 markus7017 changed the title [shelly] Various small fixes for BLU HT, Range Extender, Plus 10V, TRV set temp, event filter for alarm channel [shelly] Various small fixes for BLU HT, Range Extender, Plus 10V, TRV set temp May 28, 2024
@lsiepel
Copy link
Contributor

lsiepel commented May 28, 2024

@lolodomo I marked the issues in the initial description (Closing:) or is the syntax incorrect?

Yes, you can also see it on the right side:
image

@lsiepel lsiepel merged commit f79a2dd into openhab:main May 28, 2024
5 checks passed
@lsiepel lsiepel added this to the 4.2 milestone May 28, 2024
psmedley pushed a commit to psmedley/openhab-addons that referenced this pull request Jun 15, 2024
…V set temp (openhab#16746)

* fixes #openhab#16736, openhab#16734, openhab#16542, openhab#16709, openhab#16735

Signed-off-by: Markus Michels <markus7017@gmail.com>
Signed-off-by: Paul Smedley <paul@smedley.id.au>
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/openhab-4-2-milestone-discussion/154316/196

pgfeller pushed a commit to pgfeller/openhab-addons that referenced this pull request Sep 29, 2024
…V set temp (openhab#16746)

* fixes #openhab#16736, openhab#16734, openhab#16542, openhab#16709, openhab#16735

Signed-off-by: Markus Michels <markus7017@gmail.com>
Signed-off-by: Patrik Gfeller <patrik.gfeller@proton.me>
joni1993 pushed a commit to joni1993/openhab-addons that referenced this pull request Oct 15, 2024
matchews pushed a commit to matchews/openhab-addons that referenced this pull request Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment