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

[loxone] Implementation of EIB Dimmer #10585

Merged
merged 1 commit into from
May 9, 2021
Merged

Conversation

ppieczul
Copy link
Contributor

Requested here: ppieczul/org.openhab.binding.loxone#9

Added new class LxControlEIBDimmer, that is simpler than LxControlDimmer and becomes a parent class. LxControlDimmer made to inherit from LxControlEIBDimmer.
Added unit tests.
Fixed some of code analysis remarks.

Signed-off-by: Pawel Pieczul [email protected]

Signed-off-by: Pawel Pieczul <[email protected]>
@wborn wborn added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Apr 28, 2021
Copy link
Member

@fwolter fwolter left a comment

Choose a reason for hiding this comment

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

LGTM

@fwolter fwolter merged commit f652e32 into openhab:main May 9, 2021
@fwolter fwolter added the enhancement An enhancement or new feature for an existing add-on label May 9, 2021
@fwolter fwolter added this to the 3.1 milestone May 9, 2021
themillhousegroup pushed a commit to themillhousegroup/openhab2-addons that referenced this pull request May 10, 2021
Signed-off-by: Pawel Pieczul <[email protected]>
Signed-off-by: John Marshall <[email protected]>
computergeek1507 pushed a commit to computergeek1507/openhab-addons that referenced this pull request Jul 13, 2021
frederictobiasc pushed a commit to frederictobiasc/openhab-addons that referenced this pull request Oct 26, 2021
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
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