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
[tradfri] Added basic support for IKEA blinds FYRTUR and KADRILJ #6977
Conversation
* adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com>
Signed-off-by: Manuel Raffel <manidu@outlook.com>
- Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
"TRADFRI bulb E27 WS clear 950lm", "TRADFRI bulb GU10 WS 400lm", "TRADFRI bulb E14 WS opal 400lm", | ||
"FLOALT panel WS 30x30", "FLOALT panel WS 60x60", "FLOALT panel WS 30x90", | ||
"TRADFRI bulb E12 WS opal 400lm" }; | ||
private static final List<String> COLOR_TEMP_MODELS = Collections |
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.
Why not use a Set
here?
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.
Yes, good point. Why not use a Set
here. I changed it.
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Travis fails in itest (doesn't run at all). How do we want to handle that? I have seen that in a lot (all?) of PR lately when it comes to itests. |
At least for TRADFRI we ignored them last time (e.g. #6544). |
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com> Signed-off-by: leluna <hengrui.jiang@googlemail.com>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com> Signed-off-by: Hans-Reiner Hoffmann <hans-reiner.hoffmann@gmx.de>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com> Signed-off-by: Daan Meijer <daan@studioseptember.nl>
…nhab#6977) * Adds basic support for IKEA smart blinds FYRTUR and KADRILJ * adds the "blind" thing and associated data * temporarily assigned zigbee id "0999", needs review Signed-off-by: Manuel Raffel <manidu@outlook.com> * Restores ISO 8859-1 encoding of .properties file Signed-off-by: Manuel Raffel <manidu@outlook.com> * - Incorporated review comments - Added discovery - Added check for percent values - Aplied formatter - Fixed unit tests Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added repeater Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added TRADFRI open/close remote Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Refactoring of discovery service Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Added handling for STOP command Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> * Incorporated changes from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Co-authored-by: Manuel <manraf@users.noreply.github.com>
Supersedes #6167
Test version: org.openhab.binding.tradfri-2.5.2-SNAPSHOT.zip
Also-by: Manuel Raffel manidu@outlook.com
Signed-off-by: Christoph Weitkamp github@christophweitkamp.de