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] Added shade type 65 to database #12081

Merged

Conversation

andrewfg
Copy link
Contributor

Add Shade type 65 to database.

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 Jan 21, 2022
@andrewfg andrewfg requested a review from a team January 21, 2022 18:58
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

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

LGTM

@lolodomo lolodomo merged commit c828d81 into openhab:main Jan 21, 2022
@lolodomo lolodomo added this to the 3.3 milestone Jan 21, 2022
@jlaur
Copy link
Contributor

jlaur commented Jan 21, 2022

@andrewfg - consider if PR title should be changed to "[hdpowerview] Add full support for Vignette Duolite" or something more specific for the release notes? As I understand it, adding correct capabilities to the database will ensure that position calculations are correct, so without it defaults are used, which can lead to some undesired behavior?

@andrewfg
Copy link
Contributor Author

@jlaur I chose the title 'Update Shade Database' because there will be several such generic updates, and this is just one of many. The PR description (will) say(s) what was actually updated in each PR.

Note: this new shade type has an existing 'capabilities' value, so its functionality has already been coded, and therefore no functional changes are required now..

@lolodomo
Copy link
Contributor

I agree that a more precise title would be better.

@andrewfg andrewfg changed the title [hdpowerview] Update shade database [hdpowerview] Added shade type 65 to database Jan 22, 2022
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Jan 28, 2022
* [hdpowerview] update shade database

* [hdpowerview] add type 65 shade capabilities 8

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this pull request Apr 27, 2022
* [hdpowerview] update shade database

* [hdpowerview] add type 65 shade capabilities 8

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Signed-off-by: Nick Waterton <n.waterton@outlook.com>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
* [hdpowerview] update shade database

* [hdpowerview] add type 65 shade capabilities 8

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] update shade database

* [hdpowerview] add type 65 shade capabilities 8

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
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.

3 participants