-
Notifications
You must be signed in to change notification settings - Fork 23
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 do not disturb channel #391
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 for your first contribution over here. In general looks good, I have left two comments.
...rol/src/main/java/org/smarthomej/binding/amazonechocontrol/internal/handler/EchoHandler.java
Outdated
Show resolved
Hide resolved
...rol/src/main/java/org/smarthomej/binding/amazonechocontrol/internal/handler/EchoHandler.java
Outdated
Show resolved
Hide resolved
e9b23ad
to
f5d039b
Compare
all suggestions applied |
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.
I forgot something. We need to add the thing to each of the thing-update-lists:
2;ADD_CHANNEL;doNotDisturb,Switch,amazonechocontrol:doNotDisturb
By doing so all UI configured things will be updated with the new channel (so you don't habe to delete and re-add the thing).
@@ -206,6 +208,7 @@ | |||
<channel id="lastSpokenText" typeId="lastSpokenText"/> | |||
<channel id="notificationVolume" typeId="notificationVolume"/> | |||
<channel id="ascendingAlarm" typeId="ascendingAlarm"/> | |||
<channel id="doNotDisturb" typeId="doNotDisturb"/> | |||
</channels> | |||
<properties> | |||
<property name="thingTypeVersion">1</property> |
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.
<property name="thingTypeVersion">1</property> | |
<property name="thingTypeVersion">2</property> |
@@ -147,6 +148,7 @@ | |||
<channel id="lastSpokenText" typeId="lastSpokenText"/> | |||
<channel id="notificationVolume" typeId="notificationVolume"/> | |||
<channel id="ascendingAlarm" typeId="ascendingAlarm"/> | |||
<channel id="doNotDisturb" typeId="doNotDisturb"/> | |||
</channels> | |||
<properties> | |||
<property name="thingTypeVersion">1</property> |
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.
<property name="thingTypeVersion">1</property> | |
<property name="thingTypeVersion">2</property> |
@@ -88,6 +88,7 @@ | |||
<channel id="lastSpokenText" typeId="lastSpokenText"/> | |||
<channel id="notificationVolume" typeId="notificationVolume"/> | |||
<channel id="ascendingAlarm" typeId="ascendingAlarm"/> | |||
<channel id="doNotDisturb" typeId="doNotDisturb"/> | |||
</channels> | |||
<properties> | |||
<property name="thingTypeVersion">1</property> |
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.
<property name="thingTypeVersion">1</property> | |
<property name="thingTypeVersion">2</property> |
Signed-off-by: Cody Cutrer <cody@cutrer.us>
f5d039b
to
b96e261
Compare
oh cool, I didn't know about that. that's much better. I have 20 echos, so it's really nice to not have to recreate them all! |
Thank you very much! |
…homej#391) Signed-off-by: Cody Cutrer <cody@cutrer.us> Signed-off-by: Ronny Grun <ronny.grun@t-online.de>
Signed-off-by: Cody Cutrer cody@cutrer.us