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

[hdpowerview] Correct shade capabilities information #13330

Merged
merged 4 commits into from
Sep 10, 2022

Conversation

andrewfg
Copy link
Contributor

Background

Resolves #13320

Hunter Douglas has introduced its Generation 3 hubs. And the documentation for Gen 3 includes information (among many other things) about the Shade Capabilities. See screenshot below. Until now the OH binding uses an unofficial Capabilities database that has been based on detective work and trial and error. The Gen 3 official information confirms that 8 out of the 10 Capabilities in OH are indeed correct, but it also indicates that two of the existing Capabilities are reversed, and that one additional Capabilities entry is missing.

image

Solution

This PR swaps Capabilities 3 and 4 to match the Gen 3 official information. And it adds the so far missing Capabilities 10 the binding.

Signed-off-by: Andrew Fiddian-Green software@whitebear.ch

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
@andrewfg andrewfg added the enhancement An enhancement or new feature for an existing add-on label Aug 28, 2022
@andrewfg andrewfg self-assigned this Aug 28, 2022
@andrewfg andrewfg marked this pull request as draft August 28, 2022 16:41
@andrewfg
Copy link
Contributor Author

@jlaur I am putting this PR on GitHub, but have marked it as a draft, pending any further learnings concerning the Gen 3 hub API.

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
@andrewfg andrewfg marked this pull request as ready for review September 4, 2022 14:56
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 dce6dc9 into openhab:main Sep 10, 2022
@jlaur jlaur added this to the 3.4 milestone Sep 10, 2022
@andrewfg andrewfg deleted the #13320-hdpowerview-capabilities branch September 26, 2022 16:12
leifbladt pushed a commit to leifbladt/openhab-addons that referenced this pull request Oct 15, 2022
* [hdpowerview] corrections to capabilities database

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
* [hdpowerview] corrections to capabilities database

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
andrasU pushed a commit to andrasU/openhab-addons that referenced this pull request Nov 12, 2022
* [hdpowerview] corrections to capabilities database

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
marcelGoerentz pushed a commit to marcelGoerentz/openhab-addons that referenced this pull request Nov 14, 2022
* [hdpowerview] corrections to capabilities database

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
psmedley pushed a commit to psmedley/openhab-addons that referenced this pull request Feb 23, 2023
* [hdpowerview] corrections to capabilities database

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Feb 28, 2023
* [hdpowerview] corrections to capabilities database

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
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.

[hdpowerview] Corrections to capabilities database
2 participants