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

[deconz] support for thermostat windowopen #9306

Merged
merged 5 commits into from Dec 11, 2020
Merged

Conversation

J-N-K
Copy link
Member

@J-N-K J-N-K commented Dec 9, 2020

Closes #9233

@J-N-K J-N-K added enhancement An enhancement or new feature for an existing add-on work in progress A PR that is not yet ready to be merged labels Dec 9, 2020
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
@J-N-K J-N-K removed the work in progress A PR that is not yet ready to be merged label Dec 11, 2020
@J-N-K J-N-K changed the title [WIP][deconz] support for thermostat windowopen [deconz] support for thermostat windowopen Dec 11, 2020
Comment on lines +203 to +204
thingBuilder.withChannel(ChannelBuilder.create(new ChannelUID(thing.getUID(), CHANNEL_WINDOWOPEN), "String")
.withType(new ChannelTypeUID(BINDING_ID, "open")).build());
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't you check whether or not this channel already exists or not?

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
@cpmeister cpmeister merged commit dca815b into openhab:main Dec 11, 2020
@cpmeister cpmeister added this to the 3.0.0.RC1 milestone Dec 11, 2020
@J-N-K J-N-K deleted the deconz3 branch December 11, 2020 22:21
nowaterman pushed a commit to nowaterman/openhab-addons that referenced this pull request Jan 19, 2021
* add support for windowopen in thermostats
* fix documentation
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
* add support for windowopen in thermostats
* fix documentation

Signed-off-by: Jan N. Klug <jan.n.klug@rub.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.

[deconz] Open Window Detection as Channel for Thermostat thing
2 participants