-
-
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
[amazonechocontrol] Add channels to thermostatController #13067
[amazonechocontrol] Add channels to thermostatController #13067
Conversation
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! I have looked through your changes, and posted some feedback.
...main/java/org/openhab/binding/amazonechocontrol/internal/handler/SmartHomeDeviceHandler.java
Outdated
Show resolved
Hide resolved
.../java/org/openhab/binding/amazonechocontrol/internal/smarthome/HandlerTemperatureSensor.java
Outdated
Show resolved
Hide resolved
...va/org/openhab/binding/amazonechocontrol/internal/smarthome/HandlerThermostatController.java
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.amazonechocontrol/src/main/resources/OH-INF/thing/thing-types.xml
Outdated
Show resolved
Hide resolved
...hab.binding.amazonechocontrol/src/main/resources/OH-INF/i18n/amazonechocontrol_de.properties
Outdated
Show resolved
Hide resolved
...azonechocontrol/src/main/java/org/openhab/binding/amazonechocontrol/internal/Connection.java
Outdated
Show resolved
Hide resolved
...penhab.binding.amazonechocontrol/src/main/resources/OH-INF/i18n/amazonechocontrol.properties
Outdated
Show resolved
Hide resolved
@djcampbell - after fixing DCO and regenerating the i18n properties file, this PR can be merged. |
...penhab.binding.amazonechocontrol/src/main/resources/OH-INF/i18n/amazonechocontrol.properties
Outdated
Show resolved
Hide resolved
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
edd9aca
to
8469560
Compare
...penhab.binding.amazonechocontrol/src/main/resources/OH-INF/i18n/amazonechocontrol.properties
Outdated
Show resolved
Hide resolved
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Thank you Jacob for all the time you spent reviewing this! I truly appreciate your patience! -Daniel |
Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
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!
* enhance: add thermostat channels to amazonechocontrol Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
* enhance: add thermostat channels to amazonechocontrol Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
* enhance: add thermostat channels to amazonechocontrol Signed-off-by: Daniel Campbell <djcampbell79@gmail.com> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
* enhance: add thermostat channels to amazonechocontrol Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
* enhance: add thermostat channels to amazonechocontrol Signed-off-by: Daniel Campbell <djcampbell79@gmail.com>
Addresses issue described in #10260
Added 4 channels to amazonechocontrol:
Allows a user to change the operation mode of the Alexa Smart Thermostat
Allows a user to control upper and lower temperature thresholds when thermostat is in AUTO mode
Allows a user to view the current humidity reading from the thermostat
Full testing performed on local instance