-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[enocean] Add Ramping time to dimmer channel for central command #4585
Conversation
… Dominik Krickl-Vorreiter <[email protected]>
Signed-off-by: Dominik Krickl-Vorreiter <[email protected]>
Signed-off-by: Dominik Krickl-Vorreiter <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes are ok 👍
...an/src/main/java/org/openhab/binding/enocean/internal/config/EnOceanChannelDimmerConfig.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Dominik Krickl-Vorreiter <[email protected]>
Thank you, @wborn, I have changed the header. Let me know if I can do anything more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Can you also create a PR to update the documentation for the new thing types and configuration options @dominikkv or @fruggy83 ?
…nhab#4585) * Add ramping time to dimmer channel for central command * Add ramping time for D2-01-01 Signed-off-by: Dominik Krickl-Vorreiter <[email protected]> Signed-off-by: Pshatsillo <[email protected]>
…nhab#4585) * Add ramping time to dimmer channel for central command * Add ramping time for D2-01-01 Signed-off-by: Dominik Krickl-Vorreiter <[email protected]> Signed-off-by: Maximilian Hess <[email protected]>
Add ramping time for EEPs A5-38-08 CMD 0x02 and D2-01-01.
Signed-off-by: Dominik Krickl-Vorreiter [email protected]
closes #4584
See fruggy83/openocean#43