Merge pull request #192 from tslagle13/master

Fix requirement for SMS
This commit is contained in:
Kristofer Schaller
2015-10-20 13:42:08 -07:00
@@ -50,7 +50,7 @@ preferences {
} }
section("Send Notifications?") { section("Send Notifications?") {
input("recipients", "contact", title: "Send notifications to") { input("recipients", "contact", title: "Send notifications to") {
input "phone", "phone", title: "Send an SMS to this number?" input "phone", "phone", title: "Send an SMS to this number?", required:false
} }
} }
@@ -266,9 +266,11 @@ def sendAway(msg) {
} }
else { else {
sendPush(msg) sendPush(msg)
if(phone){
sendSms(phone, msg) sendSms(phone, msg)
} }
} }
}
log.debug(msg) log.debug(msg)
} }
@@ -280,9 +282,11 @@ def sendHome(msg) {
} }
else { else {
sendPush(msg) sendPush(msg)
if(phone){
sendSms(phone, msg) sendSms(phone, msg)
} }
} }
}
log.debug(msg) log.debug(msg)
} }