Merge pull request #1199 from SmartThingsCommunity/staging

Rolling up staging to production for deploy

devicetypes/keen-home/keen-home-smart-vent.src/keen-home-smart-vent.groovy

@@ -274,6 +274,7 @@ private Map makeTemperatureResult(value) {
         name: 'temperature',
         value: "" + value,
         descriptionText: "${linkText} is ${value}°${temperatureScale}",
+        unit: temperatureScale
     ]
 }
 

devicetypes/plaidsystems/spruce-sensor.src/spruce-sensor.groovy

@@ -254,7 +254,8 @@ private Map getTemperatureResult(value) {
 	return [
 		name: 'temperature',
 		value: value,
-		descriptionText: descriptionText
+		descriptionText: descriptionText,
+		unit: temperatureScale
 	]
 }
 

devicetypes/smartthings/bose-soundtouch.src/bose-soundtouch.groovy

@@ -47,6 +47,9 @@ metadata {
 
         command "everywhereJoin"
         command "everywhereLeave"
+
+        command "forceOff"
+        command "forceOn"
     }
 
     /**
@@ -64,9 +67,9 @@ metadata {
     }
 
     standardTile("switch", "device.switch", width: 1, height: 1, canChangeIcon: true) {
-        state "on", label: '${name}', action: "switch.off", icon: "st.Electronics.electronics16", backgroundColor: "#79b821", nextState:"turningOff"
+        state "on", label: '${name}', action: "forceOff", icon: "st.Electronics.electronics16", backgroundColor: "#79b821", nextState:"turningOff"
         state "turningOff", label:'TURNING OFF', icon:"st.Electronics.electronics16", backgroundColor:"#ffffff"
-        state "off", label: '${name}', action: "switch.on", icon: "st.Electronics.electronics16", backgroundColor: "#ffffff", nextState:"turningOn"
+        state "off", label: '${name}', action: "forceOn", icon: "st.Electronics.electronics16", backgroundColor: "#ffffff", nextState:"turningOn"
         state "turningOn", label:'TURNING ON', icon:"st.Electronics.electronics16", backgroundColor:"#79b821"
     }
     valueTile("1", "device.station1", decoration: "flat", canChangeIcon: false) {
@@ -140,8 +143,22 @@ metadata {
  * one place.
  *
  */
-def off() { onAction("off") }
-def on() { onAction("on") }
+def off() {
+    if (device.currentState("switch")?.value == "on") {
+        onAction("off")
+    }
+}
+def forceOff() {
+    onAction("off")
+}
+def on() {
+    if (device.currentState("switch")?.value == "off") {
+        onAction("on")
+    }
+}
+def forceOn() {
+    onAction("on")
+}
 def volup() { onAction("volup") }
 def voldown() { onAction("voldown") }
 def preset1() { onAction("1") }
@@ -240,11 +257,11 @@ def onAction(String user, data=null) {
     def actions = null
     switch (user) {
         case "on":
-            actions = boseSetPowerState(true)
+            boseSetPowerState(true)
             break
         case "off":
             boseSetNowPlaying(null, "STANDBY")
-            actions = boseSetPowerState(false)
+            boseSetPowerState(false)
             break
         case "volume":
             actions = boseSetVolume(data)

devicetypes/smartthings/centralite-thermostat.src/centralite-thermostat.groovy

@@ -89,14 +89,17 @@ def parse(String description) {
 			log.debug "TEMP"
 			map.name = "temperature"
 			map.value = getTemperature(descMap.value)
+			map.unit = temperatureScale
 		} else if (descMap.cluster == "0201" && descMap.attrId == "0011") {
 			log.debug "COOLING SETPOINT"
 			map.name = "coolingSetpoint"
 			map.value = getTemperature(descMap.value)
+			map.unit = temperatureScale
 		} else if (descMap.cluster == "0201" && descMap.attrId == "0012") {
 			log.debug "HEATING SETPOINT"
 			map.name = "heatingSetpoint"
 			map.value = getTemperature(descMap.value)
+			map.unit = temperatureScale
 		} else if (descMap.cluster == "0201" && descMap.attrId == "001c") {
 			log.debug "MODE"
 			map.name = "thermostatMode"
@@ -169,7 +172,7 @@ def setHeatingSetpoint(degrees) {
 
 		def degreesInteger = Math.round(degrees)
 		log.debug "setHeatingSetpoint({$degreesInteger} ${temperatureScale})"
-		sendEvent("name": "heatingSetpoint", "value": degreesInteger)
+		sendEvent("name": "heatingSetpoint", "value": degreesInteger, "unit": temperatureScale)
 
 		def celsius = (getTemperatureScale() == "C") ? degreesInteger : (fahrenheitToCelsius(degreesInteger) as Double).round(2)
 		"st wattr 0x${device.deviceNetworkId} 1 0x201 0x12 0x29 {" + hex(celsius * 100) + "}"
@@ -180,7 +183,7 @@ def setCoolingSetpoint(degrees) {
 	if (degrees != null) {
 		def degreesInteger = Math.round(degrees)
 		log.debug "setCoolingSetpoint({$degreesInteger} ${temperatureScale})"
-		sendEvent("name": "coolingSetpoint", "value": degreesInteger)
+		sendEvent("name": "coolingSetpoint", "value": degreesInteger, "unit": temperatureScale)
 		def celsius = (getTemperatureScale() == "C") ? degreesInteger : (fahrenheitToCelsius(degreesInteger) as Double).round(2)
 		"st wattr 0x${device.deviceNetworkId} 1 0x201 0x11 0x29 {" + hex(celsius * 100) + "}"
 	}

devicetypes/smartthings/cree-bulb.src/.st-ignore

@@ -0,0 +1,2 @@
+.st-ignore
+README.md

devicetypes/smartthings/cree-bulb.src/README.md

@@ -0,0 +1,36 @@
+# Connected Cree LED Bulb
+
+
+
+Works with: 
+
+* [Connected Cree LED Bulb](https://support.smartthings.com/hc/en-us/articles/204258280-Cree-Connected-LED-Bulb)
+
+## Table of contents
+
+* [Capabilities](#capabilities)
+* [Health](#device-health)
+
+## Capabilities
+
+* **Actuator** - represents that a Device has commands
+* **Configuration** - _configure()_ command called when device is installed or device preferences updated
+* **Polling** - represents that poll() can be implemented for the device
+* **Refresh** - _refresh()_ command for status updates
+* **Switch** - can detect state (possible values: on/off)
+* **Switch Level** - represents current light level, usually 0-100 in percent
+* **Health Check** - indicates ability to get device health notifications
+
+## Device Health
+
+A Category C6 Connected Cree LED Bulb with maxReportTime of 10 min.
+Check-in interval is double the value of maxReportTime for Zigbee device. 
+This gives the device twice the amount of time to respond before it is marked as offline.
+Check-in interval = 2*10 = 20 min
+
+## Troubleshooting
+
+If the device doesn't pair when trying from the SmartThings mobile app, it is possible that the device is out of range.
+Pairing needs to be tried again by placing the device closer to the hub.
+Instructions related to pairing, resetting and removing the device from SmartThings can be found in the following link:
+* [Cree Connected LED Bulb Troubleshooting Tips](https://support.smartthings.com/hc/en-us/articles/204258280-Cree-Connected-LED-Bulb)

devicetypes/smartthings/fidure-thermostat.src/fidure-thermostat.groovy

@@ -682,7 +682,7 @@ def setHeatingSetpoint(degrees) {
 	def temperatureScale = getTemperatureScale()
 
 	def degreesInteger = degrees as Integer
-	sendEvent("name":"heatingSetpoint", "value":degreesInteger)
+	sendEvent("name":"heatingSetpoint", "value":degreesInteger, "unit":temperatureScale)
 
 	def celsius = (getTemperatureScale() == "C") ? degreesInteger : (fahrenheitToCelsius(degreesInteger) as Double).round(2)
 	"st wattr 0x${device.deviceNetworkId} 1 0x201 0x12 0x29 {" + hex(celsius*100) + "}"
@@ -691,7 +691,7 @@ def setHeatingSetpoint(degrees) {
 
 def setCoolingSetpoint(degrees) {
 	def degreesInteger = degrees as Integer
-	sendEvent("name":"coolingSetpoint", "value":degreesInteger)
+	sendEvent("name":"coolingSetpoint", "value":degreesInteger, "unit":temperatureScale)
 	def celsius = (getTemperatureScale() == "C") ? degreesInteger : (fahrenheitToCelsius(degreesInteger) as Double).round(2)
 	"st wattr 0x${device.deviceNetworkId} 1 0x201 0x11 0x29 {" + hex(celsius*100) + "}"
 

devicetypes/smartthings/hue-bridge.src/hue-bridge.groovy

@@ -28,8 +28,8 @@ metadata {
 			}
 			}
 		valueTile("doNotRemove", "v", decoration: "flat", height: 2, width: 6, inactiveLabel: false) {
-			state "default", label:'Do not remove'
-        }
+			state "default", label:'If removed, Hue lights will not work properly'
+		}
 		valueTile("idNumber", "device.idNumber", decoration: "flat", height: 2, width: 6, inactiveLabel: false) {
 			state "default", label:'ID: ${currentValue}'
 		}
@@ -38,7 +38,7 @@ metadata {
 		}
 
 		main (["rich-control"])
-		details(["rich-control", "idNumber", "networkAddress", "doNotRemove"])
+		details(["rich-control", "doNotRemove", "idNumber", "networkAddress"])
 	}
 }
 
@@ -62,7 +62,7 @@ def parse(description) {
 			log.trace "HUE BRIDGE, GENERATING EVENT: $map.name: $map.value"
 			results << createEvent(name: "${map.name}", value: "${map.value}")
 		} else {
-        	log.trace "Parsing description"
+			log.trace "Parsing description"
 			def msg = parseLanMessage(description)
 			if (msg.body) {
 				def contentType = msg.headers["Content-Type"]
@@ -72,13 +72,13 @@ def parse(description) {
 						log.info "Bridge response: $msg.body"
 					} else {
 						// Sending Bulbs List to parent"
-                        if (parent.state.inBulbDiscovery)
-                        	log.info parent.bulbListHandler(device.hub.id, msg.body)
+						if (parent.isInBulbDiscovery())
+							log.info parent.bulbListHandler(device.hub.id, msg.body)
 					}
 				}
 				else if (contentType?.contains("xml")) {
 					log.debug "HUE BRIDGE ALREADY PRESENT"
-                    parent.hubVerification(device.hub.id, msg.body)
+					parent.hubVerification(device.hub.id, msg.body)
 				}
 			}
 		}

devicetypes/smartthings/nyce-motion-sensor.src/nyce-motion-sensor.groovy

@@ -147,8 +147,8 @@ private Map parseIasMessage(String description) {
 	ZoneStatus zs = zigbee.parseZoneStatus(description)
 	Map resultMap = [:]
 
-	result.name = 'motion'
-	result.value = zs.isAlarm2Set() ? 'active' : 'inactive'
+	resultMap.name = 'motion'
+	resultMap.value = zs.isAlarm2Set() ? 'active' : 'inactive'
 	log.debug(zs.isAlarm2Set() ? 'motion' : 'no motion')
 
 	return resultMap

devicetypes/smartthings/smartpower-outlet.src/.st-ignore

@@ -0,0 +1,2 @@
+.st-ignore
+README.md

devicetypes/smartthings/smartpower-outlet.src/README.md

@@ -0,0 +1,38 @@
+# SmartPower Outlet
+
+
+
+Works with: 
+
+* [Samsung SmartPower Outlet](https://shop.smartthings.com/#!/products/smartpower-outlet)
+
+## Table of contents
+
+* [Capabilities](#capabilities)
+* [Health](#device-health)
+
+## Capabilities
+
+* **Configuration** - _configure()_ command called when device is installed or device preferences updated
+* **Actuator** - represents that a Device has commands
+* **Switch** - can detect state (possible values: on/off)
+* **Refresh** - _refresh()_ command for status updates
+* **Power Meter** - detects power meter for device in either w or kw.
+* **Health Check** - indicates ability to get device health notifications
+* **Sensor** - detects sensor events
+
+## Device Health
+
+A Category C1 smart power outlet with maxReportTime of 10 min.
+Check-in interval is double the value of maxReportTime for Zigbee device. 
+This gives the device twice the amount of time to respond before it is marked as offline.
+Check-in interval = 2*10 = 20 min
+
+## Troubleshooting
+
+If the device doesn't pair when trying from the SmartThings mobile app, it is possible that the device is out of range.
+Pairing needs to be tried again by placing the device closer to the hub.
+Instructions related to pairing, resetting and removing the device from SmartThings can be found in the following links
+for the different models:
+* [SmartPower Outlet Troubleshooting Tips](https://support.smartthings.com/hc/en-us/articles/201084854-SmartPower-Outlet)
+* [Samsung SmartThings Outlet Troubleshooting Tips](https://support.smartthings.com/hc/en-us/articles/205957620)

devicetypes/smartthings/smartsense-moisture-sensor.src/.st-ignore

@@ -0,0 +1,2 @@
+.st-ignore
+README.md

devicetypes/smartthings/smartsense-moisture-sensor.src/README.md

@@ -0,0 +1,44 @@
+# Smartsense Moisture Sensor
+
+
+
+Works with: 
+
+* [Samsung SmartThings Moisture Sensor](https://shop.smartthings.com/#!/products/samsung-smartthings-water-leak-sensor)
+
+## Table of contents
+
+* [Capabilities](#capabilities)
+* [Health](#device-health)
+* [Battery](#battery-specification)
+
+## Capabilities
+
+* **Configuration** - _configure()_ command called when device is installed or device preferences updated
+* **Battery** - defines device uses a battery
+* **Refresh** - _refresh()_ command for status updates
+* **Temperature Measurement** - defines device measures current temperature
+* **Water Sensor** - can detect presence of water (dry or wet)
+* **Health Check** - indicates ability to get device health notifications
+
+## Device Health
+
+A Category C2 moisture sensor with maxReportTime of 1 hr.
+Check-in interval is double the value of maxReportTime for Zigbee device. 
+This gives the device twice the amount of time to respond before it is marked as offline.
+Check-in interval = 2*60 = 120 min
+
+## Battery Specification
+
+One CR2 3V battery required.
+
+## Troubleshooting
+
+If the sensor doesn't pair when trying from the SmartThings mobile app, it is possible that the sensor is out of range.
+Pairing needs to be tried again by placing the sensor closer to the hub.
+Instructions related to pairing, resetting and removing the different sensors from SmartThings can be found in the following links
+for the different models:
+* [SmartSense Moisture Sensor Troubleshooting Tips](https://support.smartthings.com/hc/en-us/articles/202847044-SmartSense-Moisture-Sensor)
+* [Samsung SmartThings Water Leak Sensor Troubleshooting Tips](https://support.smartthings.com/hc/en-us/articles/205957630)
+Other troubleshooting tips are listed as follows:
+* [Troubleshooting: Samsung SmartThings Water Leak Sensor won’t pair after removing pull-tab](https://support.smartthings.com/hc/en-us/articles/204966616-Troubleshooting-Samsung-SmartThings-device-won-t-pair-after-removing-pull-tab)

devicetypes/smartthings/smartsense-moisture-sensor.src/smartsense-moisture-sensor.groovy

@@ -24,7 +24,7 @@ metadata {
 		capability "Temperature Measurement"
 		capability "Water Sensor"
 		capability "Health Check"
-		capability "Sensor"		
+		capability "Sensor"
 
 		command "enrollResponse"
 
@@ -259,7 +259,8 @@ private Map getTemperatureResult(value) {
 		name: 'temperature',
 		value: value,
 		descriptionText: descriptionText,
-        translatable: true
+		translatable: true,
+		unit: temperatureScale
 	]
 }
 
@@ -304,7 +305,7 @@ def refresh() {
 }
 
 def configure() {
-	sendEvent(name: "checkInterval", value: 7200, displayed: false, data: [protocol: "zigbee"])
+	sendEvent(name: "checkInterval", value: 14400, displayed: false, data: [protocol: "zigbee"])
 
 	String zigbeeEui = swapEndianHex(device.hub.zigbeeEui)
 	log.debug "Configuring Reporting, IAS CIE, and Bindings."

devicetypes/smartthings/smartsense-motion-sensor.src/README.md

@@ -9,7 +9,8 @@ Works with:
 ## Table of contents
 
 * [Capabilities](#capabilities)
-* [Health]($health)
+* [Health](#device-health)
+* [Battery](#battery-specification)
 
 ## Capabilities
 
@@ -21,10 +22,24 @@ Works with:
 
 ## Device Health
 
-A Category C2 motion sensor that has 120min check-in interval
-
-
+A Category C2 motion sensor with maxReportTime of 1 hr.
+Check-in interval is double the value of maxReportTime for Zigbee device. 
+This gives the device twice the amount of time to respond before it is marked as offline.
+Check-in interval = 2*60 = 120 min
 
+## Battery Specification
 
+One CR2477 (for Samsung SmartThings Motion Sensor) / CR123A (SmartSense Motion Sensor) 3V battery is required.
 
+## Troubleshooting
 
+If the device doesn't pair when trying from the SmartThings mobile app, it is possible that the sensor is out of range.
+Pairing needs to be tried again by placing the sensor closer to the hub.
+Instructions related to pairing, resetting and removing the different motion sensors from SmartThings can be found in the following links
+for the different models:
+* [SmartSense Motion Sensor (original model) Troubleshooting Tips](https://support.smartthings.com/hc/en-us/articles/200903280-SmartSense-Motion-Sensor-original-model-)
+* [SmartSense Motion Sensor (2014 model) Troubleshooting Tips](https://support.smartthings.com/hc/en-us/articles/203077520-SmartSense-Motion-Sensor-2014-model-)
+* [Samsung SmartThings Motion Sensor (2015 model) Troubleshooting Tips](https://support.smartthings.com/hc/en-us/articles/205957580-Samsung-SmartThings-Motion-Sensor-2015-model-)
+Other troubleshooting tips are listed as follows:
+* [Troubleshooting: Samsung SmartThings Motion Sensor is stuck showing "Motion Detected" or "No Motion"](https://support.smartthings.com/hc/en-us/articles/200961130-Troubleshooting-Samsung-SmartThings-Motion-Sensor-is-stuck-showing-Motion-Detected-or-No-Motion-)
+* [Troubleshooting: Samsung SmartThings Motion Sensor won’t pair after removing pull-tab](https://support.smartthings.com/hc/en-us/articles/204966616-Troubleshooting-Samsung-SmartThings-device-won-t-pair-after-removing-pull-tab)

devicetypes/smartthings/smartsense-motion-sensor.src/smartsense-motion-sensor.groovy

@@ -24,7 +24,7 @@ metadata {
 		capability "Temperature Measurement"
 		capability "Refresh"
 		capability "Health Check"
-		capability "Sensor"		
+		capability "Sensor"
 
 		command "enrollResponse"
 
@@ -274,7 +274,8 @@ private Map getTemperatureResult(value) {
 		name: 'temperature',
 		value: value,
 		descriptionText: descriptionText,
-        translatable: true
+		translatable: true,
+		unit: temperatureScale
 	]
 }
 
@@ -315,7 +316,7 @@ def refresh() {
 }
 
 def configure() {
-	sendEvent(name: "checkInterval", value: 7200, displayed: false, data: [protocol: "zigbee"])
+	sendEvent(name: "checkInterval", value: 14400, displayed: false, data: [protocol: "zigbee"])
 
 	String zigbeeEui = swapEndianHex(device.hub.zigbeeEui)
 	log.debug "Configuring Reporting, IAS CIE, and Bindings."

devicetypes/smartthings/smartsense-motion-temp-sensor.src/smartsense-motion-temp-sensor.groovy

@@ -170,7 +170,7 @@ private Map parseCustomMessage(String description) {
 
 private Map parseIasMessage(String description) {
 	ZoneStatus zs = zigbee.parseZoneStatus(description)
-	return zs.isAlarm1Set() ? getMotionResult('active') : getMotionResult('inactive')
+	return (zs.isAlarm1Set() || zs.isAlarm2Set()) ? getMotionResult('active') : getMotionResult('inactive')
 }
 
 def getTemperature(value) {
@@ -226,7 +226,8 @@ private Map getTemperatureResult(value) {
 	return [
 		name: 'temperature',
 		value: value,
-		descriptionText: descriptionText
+		descriptionText: descriptionText,
+		unit: temperatureScale
 	]
 }
 

devicetypes/smartthings/smartsense-multi-sensor.src/.st-ignore

@@ -0,0 +1,2 @@
+.st-ignore
+README.md

devicetypes/smartthings/smartsense-multi-sensor.src/README.md

@@ -0,0 +1,45 @@
+# Smartsense Multi Sensor
+
+
+
+Works with: 
+
+* [Samsung SmartThings Multi Sensor](https://shop.smartthings.com/#!/products/smartsense-multi)
+
+## Table of contents
+
+* [Capabilities](#capabilities)
+* [Health](#device-health)
+* [Battery](#battery-specification)
+
+## Capabilities
+
+* **Three Axis** - monitors the state of a single axis
+* **Configuration** - _configure()_ command called when device is installed or device preferences updated
+* **Battery** - defines device uses a battery
+* **Sensor** - detects sensor events
+* **Contact Sensor** - can detect contact (possible values: open,closed)
+* **Acceleration Sensor** - allows for acceleration detection.
+* **Refresh** - _refresh()_ command for status updates
+* **Temperature Measurement** - defines device measures current temperature
+* **Health Check** - indicates ability to get device health notifications
+
+## Device Health
+
+A Category C2 multi sensor with maxReportTime of 1 hr.
+Check-in interval is double the value of maxReportTime for Zigbee device. 
+This gives the device twice the amount of time to respond before it is marked as offline.
+Check-in interval = 2*60 = 120 min
+
+## Battery Specification
+
+One CR2450 (for Samsung SmartThings Multipurpose Sensor) battery / Two AAAA (for SmartSense Multi Sensor) batteries required.
+
+## Troubleshooting
+
+If the sensor doesn't pair when trying from the SmartThings mobile app, it is possible that the sensor is out of range.
+Pairing needs to be tried again by placing the sensor closer to the hub.
+Other troubleshooting tips are listed as follows:
+* [Troubleshooting: Samsung SmartThings Multipurpose Sensor is stuck on "open" or "closed"](https://support.smartthings.com/hc/en-us/articles/200955940-Troubleshooting-Samsung-SmartThings-Multipurpose-Sensor-is-stuck-on-open-or-closed-)
+* [Troubleshooting: Temperature reading for the Samsung SmartThings Multipurpose Sensor is off](https://support.smartthings.com/hc/en-us/articles/200756845-Troubleshooting-Temperature-reading-for-the-Samsung-SmartThings-Multipurpose-Sensor-is-off)
+* [Troubleshooting: Samsung SmartThings Multipurpose Sensor won’t pair after removing pull-tab](https://support.smartthings.com/hc/en-us/articles/204966616-Troubleshooting-Samsung-SmartThings-device-won-t-pair-after-removing-pull-tab)

devicetypes/smartthings/smartsense-multi-sensor.src/smartsense-multi-sensor.groovy

@@ -333,10 +333,11 @@ private Map getTemperatureResult(value) {
 			'{{ device.displayName }} was {{ value }}°F'
 
 	return [
-	name: 'temperature',
-	value: value,
-	descriptionText: descriptionText,
-    translatable: true
+		name: 'temperature',
+		value: value,
+		descriptionText: descriptionText,
+		translatable: true,
+		unit: temperatureScale
 	]
 }
 
@@ -413,7 +414,7 @@ def refresh() {
 }
 
 def configure() {
-	sendEvent(name: "checkInterval", value: 7200, displayed: false, data: [protocol: "zigbee"])
+	sendEvent(name: "checkInterval", value: 14400, displayed: false, data: [protocol: "zigbee"])
 
 	log.debug "Configuring Reporting"
 

devicetypes/smartthings/smartsense-open-closed-accelerometer-sensor.src/smartsense-open-closed-accelerometer-sensor.groovy

@@ -223,9 +223,10 @@ def getTemperature(value) {
 		}
 		def descriptionText = "${linkText} was ${value}°${temperatureScale}"
 		return [
-		name: 'temperature',
-		value: value,
-		descriptionText: descriptionText
+			name: 'temperature',
+			value: value,
+			descriptionText: descriptionText,
+			unit: temperatureScale
 		]
 	}
 

devicetypes/smartthings/smartsense-open-closed-sensor.src/.st-ignore

@@ -0,0 +1,2 @@
+.st-ignore
+README.md

devicetypes/smartthings/smartsense-open-closed-sensor.src/README.md

@@ -0,0 +1,41 @@
+# Smartsense Open/Closed Sensor
+
+
+
+Works with: 
+
+* [Samsung SmartThings Open/Closed Sensor](https://shop.smartthings.com/#!/packs/smartsense-open-closed-sensor/)
+
+## Table of contents
+
+* [Capabilities](#capabilities)
+* [Health](#device-health)
+* [Battery](#battery-specification)
+
+## Capabilities
+
+* **Configuration** - _configure()_ command called when device is installed or device preferences updated
+* **Battery** - defines device uses a battery
+* **Contact Sensor** - can detect contact (possible values: open,closed)
+* **Refresh** - _refresh()_ command for status updates
+* **Temperature Measurement** - defines device measures current temperature
+* **Health Check** - indicates ability to get device health notifications
+* **Sensor** - detects sensor events
+
+## Device Health
+
+A Category C2 open/closed sensor with maxReportTime of 1 hr.
+Check-in interval is double the value of maxReportTime for Zigbee device. 
+This gives the device twice the amount of time to respond before it is marked as offline.
+Check-in interval = 2*60 = 120 min
+
+## Battery Specification
+
+One CR2 3V battery required.
+
+## Troubleshooting
+
+If the sensor doesn't pair when trying from the SmartThings mobile app, it is possible that the sensor is out of range.
+Pairing needs to be tried again by placing the sensor closer to the hub.
+Instructions related to pairing, resetting and removing the sensor from SmartThings can be found in the following link:
+* [SmartSense Open/Closed Sensor Troubleshooting Tips](https://support.smartthings.com/hc/en-us/articles/202836844-SmartSense-Open-Closed-Sensor)

devicetypes/smartthings/smartsense-open-closed-sensor.src/smartsense-open-closed-sensor.groovy

@@ -226,7 +226,8 @@ private Map getTemperatureResult(value) {
 	return [
 		name: 'temperature',
 		value: value,
-		descriptionText: descriptionText
+		descriptionText: descriptionText,
+		unit: temperatureScale
 	]
 }
 
@@ -267,7 +268,7 @@ def refresh() {
 }
 
 def configure() {
-	sendEvent(name: "checkInterval", value: 7200, displayed: false, data: [protocol: "zigbee"])
+	sendEvent(name: "checkInterval", value: 14400, displayed: false, data: [protocol: "zigbee"])
 
 	String zigbeeEui = swapEndianHex(device.hub.zigbeeEui)
 	log.debug "Configuring Reporting, IAS CIE, and Bindings."

devicetypes/smartthings/smartsense-temp-humidity-sensor.src/.st-ignore

@@ -0,0 +1,2 @@
+.st-ignore
+README.md

devicetypes/smartthings/smartsense-temp-humidity-sensor.src/README.md

@@ -0,0 +1,41 @@
+# SmartSense Temp/Humidity Sensor
+
+
+
+Works with: 
+
+* [Samsung SmartSense Temp/Humidity Sensor](https://shop.smartthings.com/#!/products/smartsense-temp-humidity-sensor)
+
+## Table of contents
+
+* [Capabilities](#capabilities)
+* [Health](#device-health)
+* [Battery](#battery-specification)
+
+## Capabilities
+
+* **Configuration** - _configure()_ command called when device is installed or device preferences updated
+* **Battery** - defines device uses a battery
+* **Relative Humidity Measurement** - defines device measures relative humidity
+* **Refresh** - _refresh()_ command for status updates
+* **Temperature Measurement** - defines device measures current temperature
+* **Health Check** - indicates ability to get device health notifications
+* **Sensor** - detects sensor events
+
+## Device Health
+
+A Category C2 SmartSense Temp/Humidity Sensor with maxReportTime of 1 hr.
+Check-in interval is double the value of maxReportTime for Zigbee device. 
+This gives the device twice the amount of time to respond before it is marked as offline.
+Check-in interval = 2*60 = 120 min
+
+## Battery Specification
+
+One CR2 battery is required.
+
+## Troubleshooting
+
+If the sensor doesn't pair when trying from the SmartThings mobile app, it is possible that the sensor is out of range.
+Pairing needs to be tried by placing the sensor closer to the hub.
+Instructions related to pairing, resetting and removing the sensor from SmartThings can be found in the following link:
+* [Troubleshooting Tips](https://support.smartthings.com/hc/en-us/articles/203040294)

devicetypes/smartthings/smartsense-temp-humidity-sensor.src/smartsense-temp-humidity-sensor.groovy

@@ -233,7 +233,8 @@ private Map getTemperatureResult(value) {
 	return [
 		name: 'temperature',
 		value: value,
-		descriptionText: descriptionText
+		descriptionText: descriptionText,
+		unit: temperatureScale
 	]
 }
 
@@ -275,7 +276,7 @@ def refresh()
 }
 
 def configure() {
-	sendEvent(name: "checkInterval", value: 7200, displayed: false, data: [protocol: "zigbee"])
+	sendEvent(name: "checkInterval", value: 14400, displayed: false, data: [protocol: "zigbee"])
 
 	log.debug "Configuring Reporting and Bindings."
 	def configCmds = [

devicetypes/smartthings/tyco-door-window-sensor.src/tyco-door-window-sensor.groovy

@@ -213,7 +213,8 @@ private Map getTemperatureResult(value) {
 	return [
 		name: 'temperature',
 		value: value,
-		descriptionText: descriptionText
+		descriptionText: descriptionText,
+		unit: temperatureScale
 	]
 }
 

devicetypes/smartthings/zigbee-rgbw-bulb.src/zigbee-rgbw-bulb.groovy

@@ -106,11 +106,11 @@ def parse(String description) {
         if (zigbeeMap?.clusterInt == COLOR_CONTROL_CLUSTER) {
             if(zigbeeMap.attrInt == ATTRIBUTE_HUE){  //Hue Attribute
                 def hueValue = Math.round(zigbee.convertHexToInt(zigbeeMap.value) / 255 * 360)
-                sendEvent(name: "hue", value: hueValue, displayed:false)
+                sendEvent(name: "hue", value: hueValue, descriptionText: "Color has changed")
             }
             else if(zigbeeMap.attrInt == ATTRIBUTE_SATURATION){ //Saturation Attribute
                 def saturationValue = Math.round(zigbee.convertHexToInt(zigbeeMap.value) / 255 * 100)
-                sendEvent(name: "saturation", value: saturationValue, displayed:false)
+                sendEvent(name: "saturation", value: saturationValue, descriptionText: "Color has changed", displayed: false)
             }
         }
         else {

devicetypes/smartthings/zigbee-white-color-temperature-bulb.src/zigbee-white-color-temperature-bulb.groovy

@@ -22,6 +22,7 @@ metadata {
         capability "Actuator"
         capability "Color Temperature"
         capability "Configuration"
+        capability "Health Check"
         capability "Refresh"
         capability "Switch"
         capability "Switch Level"
@@ -35,6 +36,7 @@ metadata {
         fingerprint profileId: "0104", inClusters: "0000, 0003, 0004, 0005, 0006, 0008, 0300, 0B04, FC0F", outClusters: "0019", manufacturer: "OSRAM", model: "LIGHTIFY RT Tunable White", deviceJoinName: "OSRAM LIGHTIFY LED Recessed Kit RT 5/6 Tunable White"
         fingerprint profileId: "0104", inClusters: "0000, 0003, 0004, 0005, 0006, 0008, 0300, 0B04, FC0F", outClusters: "0019", manufacturer: "OSRAM", model: "Classic A60 TW", deviceJoinName: "OSRAM LIGHTIFY LED Tunable White 60W"
         fingerprint profileId: "0104", inClusters: "0000, 0003, 0004, 0005, 0006, 0008, 0300, 0B04, FC0F", outClusters: "0019", manufacturer: "OSRAM", model: "LIGHTIFY A19 Tunable White", deviceJoinName: "OSRAM LIGHTIFY LED Tunable White 60W"
+        fingerprint profileId: "0104", inClusters: "0000, 0003, 0004, 0005, 0006, 0008, 0300, 0B04, FC0F", outClusters: "0019", manufacturer: "OSRAM", model: "Classic B40 TW - LIGHTIFY", deviceJoinName: "OSRAM LIGHTIFY Classic B40 Tunable White"
     }
 
     // UI tile definitions
@@ -72,6 +74,12 @@ def parse(String description) {
     log.debug "description is $description"
     def event = zigbee.getEvent(description)
     if (event) {
+        // Temporary fix for the case when Device is OFFLINE and is connected again
+        if (state.lastActivity == null){
+            state.lastActivity = now()
+            sendEvent(name: "deviceWatch-lastActivity", value: state.lastActivity, description: "Last Activity is on ${new Date((long)state.lastActivity)}", displayed: false, isStateChange: true)
+        }
+        state.lastActivity = now()
         if (event.name=="level" && event.value==0) {}
         else {
             if (event.name=="colorTemperature") {
@@ -98,12 +106,29 @@ def setLevel(value) {
     zigbee.setLevel(value)
 }
 
+/**
+ * PING is used by Device-Watch in attempt to reach the Device
+ * */
+def ping() {
+
+    if (state.lastActivity < (now() - (1000 * device.currentValue("checkInterval"))) ){
+        log.info "ping, alive=no, lastActivity=${state.lastActivity}"
+        state.lastActivity = null
+        return zigbee.onOffRefresh()
+    } else {
+        log.info "ping, alive=yes, lastActivity=${state.lastActivity}"
+        sendEvent(name: "deviceWatch-lastActivity", value: state.lastActivity, description: "Last Activity is on ${new Date((long)state.lastActivity)}", displayed: false, isStateChange: true)
+    }
+}
+
 def refresh() {
     zigbee.onOffRefresh() + zigbee.levelRefresh() + zigbee.colorTemperatureRefresh() + zigbee.onOffConfig() + zigbee.levelConfig() + zigbee.colorTemperatureConfig()
 }
 
 def configure() {
     log.debug "Configuring Reporting and Bindings."
+    // Enrolls device to Device-Watch with 3 x Reporting interval 30min
+    sendEvent(name: "checkInterval", value: 1800, displayed: false, data: [protocol: "zigbee"])
     zigbee.onOffConfig() + zigbee.levelConfig() + zigbee.colorTemperatureConfig() + zigbee.onOffRefresh() + zigbee.levelRefresh() + zigbee.colorTemperatureRefresh()
 }
 

smartapps/citrix/octoblu.src/octoblu.groovy

@@ -1,927 +0,0 @@
-/**
-The MIT License (MIT)
-Copyright (c) 2016 Octoblu
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.
-*/
-
-import org.apache.commons.codec.binary.Base64
-import java.text.DecimalFormat
-import groovy.transform.Field
-
-@Field final USE_DEBUG = true
-@Field final selectedCapabilities = [ "actuator", "sensor" ]
-
-private getVendorName()       { "Octoblu" }
-private getVendorIcon()       { "http://i.imgur.com/BjTfDYk.png" }
-private apiUrl()              { appSettings.apiUrl ?: "https://meshblu.octoblu.com/" }
-private getVendorAuthPath()   { appSettings.vendorAuthPath ?: "https://oauth.octoblu.com/authorize" }
-private getVendorTokenPath()  { appSettings.vendorTokenPath ?: "https://oauth.octoblu.com/access_token" }
-
-definition(
-name: "Octoblu",
-namespace: "citrix",
-author: "Octoblu",
-description: "Connect SmartThings devices to Octoblu",
-category: "SmartThings Labs",
-iconUrl: "http://i.imgur.com/BjTfDYk.png",
-iconX2Url: "http://i.imgur.com/BjTfDYk.png"
-) {
-  appSetting "apiUrl"
-  appSetting "vendorAuthPath"
-  appSetting "vendorTokenPath"
-}
-
-preferences {
-  page(name: "welcomePage")
-  page(name: "authPage")
-  page(name: "subscribePage")
-  page(name: "devicesPage")
-}
-
-mappings {
-  path("/oauthCode") {
-    action: [ GET: "getOauthCode" ]
-  }
-  path("/message") {
-    action: [ POST: "postMessage" ]
-  }
-  path("/app") {
-    action: [ POST: "postApp" ]
-  }
-}
-
-// --------------------------------------
-
-def getDevInfo() {
-  return state.vendorDevices.collect { k, v -> "${v.uuid} " }.sort().join(" \n")
-}
-
-// --------------------------------------
-
-def welcomePage() {
-  cleanUpTokens()
-
-  return dynamicPage(name: "welcomePage", nextPage: "authPage", uninstall: showUninstall) {
-    section {
-      paragraph title: "Welcome to the Octoblu SmartThings App!", "press 'Next' to continue"
-    }
-    if (state.vendorDevices && state.vendorDevices.size()>0) {
-      section {
-        paragraph title: "My SmartThings in Octobu (${state.vendorDevices.size()}):", getDevInfo()
-      }
-    }
-    if (state.installed) {
-      section {
-        input name: "showUninstall", type: "bool", title: "Uninstall", submitOnChange: true
-        if (showUninstall) {
-          state.removeDevices = removeDevices
-          input name: "removeDevices", type: "bool", title: "Remove Octoblu devices", submitOnChange: true
-          paragraph title: "Sorry to see you go!", "please email <support@octoblu.com> with any feedback or issues"
-        }
-      }
-    }
-  }
-}
-
-// --------------------------------------
-
-def authPage() {
-
-  if (!state.accessToken) {
-    createAccessToken()
-  }
-
-  debug "using app access token ${state.accessToken}"
-
-  if (!state.vendorOAuthToken) {
-    createOAuthDevice()
-  }
-
-  def oauthParams = [
-    response_type: "code",
-    client_id: state.vendorOAuthUuid,
-    redirect_uri: getApiServerUrl() + "/api/token/${state.accessToken}/smartapps/installations/${app.id}/oauthCode"
-  ]
-
-  def redirectUrl =  getVendorAuthPath() + '?' + toQueryString(oauthParams)
-  debug "tokened redirect_uri = ${oauthParams.redirect_uri}"
-
-  def isRequired = !state.vendorBearerToken
-  return dynamicPage(name: "authPage", title: "Octoblu Authentication", nextPage:(isRequired ? null : "subscribePage"), install: isRequired) {
-    section {
-      debug "url: ${redirectUrl}"
-      if (isRequired) {
-        href url:redirectUrl, style:"embedded", title: "Authorize with Octoblu", required: isRequired, description:"please login with Octoblu to complete setup"
-      } else {
-        paragraph title: "Please press 'Next' to continue", "Octoblu token has been created"
-      }
-    }
-  }
-}
-
-def createOAuthDevice() {
-  def oAuthDevice = [
-    "name": "SmartThings",
-    "owner": "68c39f40-cc13-4560-a68c-e8acd021cff9",
-    "type": "device:oauth",
-    "online": true,
-    "options": [
-      "name": "SmartThings",
-      "imageUrl": "https://i.imgur.com/TsXefbK.png",
-      "callbackUrl": getApiServerUrl() + "/api"
-    ],
-    "configureWhitelist": [ "68c39f40-cc13-4560-a68c-e8acd021cff9" ],
-    "discoverWhitelist": [ "*" ],
-    "receiveWhitelist": [],
-    "sendWhitelist": []
-  ]
-
-  def postParams = [ uri: apiUrl()+"devices",
-  body: groovy.json.JsonOutput.toJson(oAuthDevice)]
-
-  try {
-    httpPostJson(postParams) { response ->
-      debug "got new token for oAuth device ${response.data}"
-      state.vendorOAuthUuid = response.data.uuid
-      state.vendorOAuthToken = response.data.token
-    }
-  } catch (e) {
-    log.error "unable to create oAuth device: ${e}"
-  }
-
-}
-
-// --------------------------------------
-
-def subscribePage() {
-  return dynamicPage(name: "subscribePage", title: "Subscribe to SmartThing devices", nextPage: "devicesPage") {
-    section {
-      // input name: "selectedCapabilities", type: "enum", title: "capability filter",
-      // submitOnChange: true, multiple: true, required: false, options: [ "actuator", "sensor" ]
-      for (capability in selectedCapabilities) {
-         input name: "${capability}Capability".toString(), type: "capability.$capability", title: "${capability.capitalize()} Things", multiple: true, required: false
-      }
-    }
-    section(" ") {
-      input name: "pleaseCreateAppDevice", type: "bool", title: "Create a SmartApp device", defaultValue: true
-      paragraph "A SmartApp device allows access to location and hub information for this installation"
-    }
-    section(" ") {
-      paragraph title: "", "Existing Octoblu devices may be modified!"
-    }
-  }
-}
-
-// --------------------------------------
-
-def devicesPage() {
-  def postParams = [
-    uri: apiUrl() + "devices?owner=${state.vendorUuid}&category=smart-things",
-    headers: ["Authorization": "Bearer ${state.vendorBearerToken}"]
-  ]
-  state.vendorDevices = [:]
-
-  def hasDevice = [:]
-  hasDevice[app.id] = true
-  selectedCapabilities.each { capability ->
-    def smartDevices = settings["${capability}Capability"]
-    smartDevices.each { smartDevice ->
-      hasDevice[smartDevice.id] = true
-    }
-  }
-
-  debug "getting url ${postParams.uri}"
-  try {
-    httpGet(postParams) { response ->
-      debug "devices json ${response.data.devices}"
-      response.data.devices.each { device ->
-        if (device.smartDeviceId && hasDevice[device.smartDeviceId]) {
-          debug "found device ${device.uuid} with smartDeviceId ${device.smartDeviceId}"
-          state.vendorDevices[device.smartDeviceId] = getDeviceInfo(device)
-        }
-        debug "has device: ${device.uuid} ${device.name} ${device.type}"
-      }
-    }
-  } catch (e) {
-    log.error "devices error ${e}"
-  }
-
-  selectedCapabilities.each { capability ->
-    debug "checking devices for capability ${capability}"
-    createDevices(settings["${capability}Capability"])
-  }
-  if (pleaseCreateAppDevice)
-    createAppDevice()
-
-  return dynamicPage(name: "devicesPage", title: "Octoblu Things", install: true) {
-    section {
-      paragraph title: "Please press 'Done' to finish setup", "and subscribe to SmartThing events"
-      paragraph title: "My Octoblu UUID:", "${state.vendorUuid}"
-      paragraph title: "My SmartThings in Octobu (${state.vendorDevices.size()}):", getDevInfo()
-    }
-  }
-}
-
-def createDevices(smartDevices) {
-
-  smartDevices.each { smartDevice ->
-    def commands = [
-      [ "name": "app-get-value" ],
-      [ "name": "app-get-state" ],
-      [ "name": "app-get-device" ],
-      [ "name": "app-get-events" ]
-    ]
-
-    smartDevice.supportedCommands.each { command ->
-      if (command.arguments.size()>0) {
-        commands.push([ "name": command.name, "args": command.arguments ])
-      } else {
-        commands.push([ "name": command.name ])
-      }
-    }
-
-    debug "creating device for ${smartDevice.id}"
-
-    def schemas = [
-      "version": "2.0.0",
-      "message": [:]
-    ]
-
-    commands.each { command ->
-      schemas."message"."$command.name" = [
-        "type": "object",
-        "properties": [
-          "smartDeviceId": [
-            "type": "string",
-            "readOnly": true,
-            "default": "$smartDevice.id",
-            "x-schema-form": [
-            	"condition": "false"
-            ]
-          ],
-          "command": [
-            "type": "string",
-            "readOnly": true,
-            "default": "$command.name",
-            "enum": ["$command.name"],
-            "x-schema-form": [
-            	"condition": "false"
-            ]
-          ]
-        ]
-      ]
-
-      if (command.args) {
-        schemas."message"."$command.name"."properties"."args" = [
-          "type": "object",
-          "title": "Arguments",
-          "properties": [:]
-        ]
-
-        command.args.each { arg ->
-          def argLower = "$arg"
-          argLower = argLower.toLowerCase()
-          if (argLower == "color_map") {
-            schemas."message"."$command.name"."properties"."args"."properties"."$argLower" = [
-              "type": "object",
-              "properties": [
-                "hex": [
-                  "type": "string"
-                ],
-                "level": [
-                  "type": "number"
-                ]
-              ]
-            ]
-          } else {
-            schemas."message"."$command.name"."properties"."args"."properties"."$argLower" = [
-              "type": "$argLower"
-            ]
-          }
-        }
-      }
-    }
-
-    debug "UPDATED message schema: ${schemas}"
-
-    def deviceProperties = [
-      "schemas": schemas,
-      "needsSetup": false,
-      "online": true,
-      "name": "${smartDevice.displayName}",
-      "smartDeviceId": "${smartDevice.id}",
-      "logo": "https://i.imgur.com/TsXefbK.png",
-      "owner": "${state.vendorUuid}",
-      "configureWhitelist": [],
-      "discoverWhitelist": ["${state.vendorUuid}"],
-      "receiveWhitelist": [],
-      "sendWhitelist": [],
-      "type": "device:${smartDevice.name.replaceAll('\\s','-').toLowerCase()}",
-      "category": "smart-things",
-      "meshblu": [
-        "forwarders": [
-          "received": [[
-            "url": getApiServerUrl() + "/api/token/${state.accessToken}/smartapps/installations/${app.id}/message",
-            "method": "POST",
-            "type": "webhook"
-          ]]
-        ]
-      ]
-    ]
-
-    updatePermissions(deviceProperties, smartDevice.id)
-    def params = [
-      uri: apiUrl() + "devices",
-      headers: ["Authorization": "Bearer ${state.vendorBearerToken}"],
-      body: groovy.json.JsonOutput.toJson(deviceProperties)
-    ]
-
-    try {
-
-      if (!state.vendorDevices[smartDevice.id]) {
-        debug "creating new device for ${smartDevice.id}"
-        httpPostJson(params) { response ->
-          state.vendorDevices[smartDevice.id] = getDeviceInfo(response.data)
-        }
-        return
-      }
-
-      params.uri = params.uri + "/${state.vendorDevices[smartDevice.id].uuid}"
-      debug "the device ${smartDevice.id} has already been created, updating ${params.uri}"
-      httpPutJson(params) { response ->
-        resetVendorDeviceToken(smartDevice.id);
-      }
-
-    } catch (e) {
-      log.error "unable to create new device ${e}"
-    }
-  }
-}
-
-def createAppDevice() {
-  def commands = [
-    [ "name": "app-get-location" ],
-    [ "name": "app-get-devices" ],
-    [ "name": "app-set-mode" ],
-  ]
-
-  def schemas = [
-    "version": "2.0.0",
-    "message": [:]
-  ]
-
-  commands.each { command ->
-    schemas."message"."$command.name" = [
-      "type": "object",
-      "properties": [
-        "command": [
-          "type": "string",
-          "readOnly": true,
-          "default": "$command.name",
-          "enum": ["$command.name"],
-          "x-schema-form": [
-            "condition": "false"
-          ]
-        ]
-      ]
-    ]
-  }
-
-  schemas."message"."app-set-mode"."properties"."args" = [
-    "type": "object",
-    "title": "Arguments",
-    "properties": [
-      "mode": [
-        "type": "string"
-      ]
-    ]
-  ]
-
-  def deviceProperties = [
-    "schemas": schemas,
-    "needsSetup": false,
-    "online": true,
-    "name": "${location.name} SmartApp",
-    "smartDeviceId": "${app.id}",
-    "logo": "https://i.imgur.com/TsXefbK.png",
-    "owner": "${state.vendorUuid}",
-    "configureWhitelist": [],
-    "discoverWhitelist": ["${state.vendorUuid}"],
-    "receiveWhitelist": [],
-    "sendWhitelist": [],
-    "type": "device:smart-things-app",
-    "category": "smart-things",
-    "meshblu": [
-      "forwarders": [
-        "received": [[
-          "url": getApiServerUrl() + "/api/token/${state.accessToken}/smartapps/installations/${app.id}/app",
-          "method": "POST",
-          "type": "webhook"
-        ]]
-      ]
-    ]
-  ]
-
-  updatePermissions(deviceProperties, app.id)
-  def params = [
-    uri: apiUrl() + "devices",
-    headers: ["Authorization": "Bearer ${state.vendorBearerToken}"],
-    body: groovy.json.JsonOutput.toJson(deviceProperties)
-  ]
-
-  debug "creating app device!"
-  debug params.body
-  try {
-
-    // debug params
-    if (!state.vendorDevices[app.id]) {
-      debug "creating new app device for ${app.id}"
-      httpPostJson(params) { response ->
-        state.vendorDevices[app.id] = getDeviceInfo(response.data)
-      }
-      return
-    }
-
-    params.uri = params.uri + "/${state.vendorDevices[app.id].uuid}"
-    debug "the app device ${app.id} has already been created, updating ${params.uri}"
-    httpPutJson(params) { response ->
-      resetVendorDeviceToken(app.id);
-    }
-
-  } catch (e) {
-    log.error "unable to create new device ${e}"
-  }
-
-}
-
-def updatePermissions(newDevice, id) {
-  def device = state.vendorDevices[id]
-  if (!device) return
-  newDevice.configureWhitelist = device.configureWhitelist
-  newDevice.discoverWhitelist = device.discoverWhitelist
-  newDevice.receiveWhitelist = device.receiveWhitelist
-  newDevice.sendWhitelist = device.sendWhitelist
-}
-
-def getDeviceInfo(device) {
-  return [
-    "uuid": device.uuid,
-    "token": device.token,
-    "configureWhitelist": device.configureWhitelist,
-    "discoverWhitelist": device.discoverWhitelist,
-    "receiveWhitelist": device.receiveWhitelist,
-    "sendWhitelist": device.sendWhitelist,
-  ]
-}
-
-def resetVendorDeviceToken(smartDeviceId) {
-  def deviceUUID = state.vendorDevices[smartDeviceId].uuid
-  if (!deviceUUID) {
-    debug "no device uuid in resetVendorDeviceToken?"
-    return
-  }
-  debug "getting new token for ${smartDeviceId}/${deviceUUID}"
-  def postParams = [
-  uri: apiUrl() + "devices/${deviceUUID}/token",
-  headers: ["Authorization": "Bearer ${state.vendorBearerToken}"]]
-  try {
-    httpPost(postParams) { response ->
-      state.vendorDevices[smartDeviceId] = getDeviceInfo(response.data)
-      debug "got new token for ${smartDeviceId}/${deviceUUID}"
-    }
-  } catch (e) {
-    log.error "unable to get new token ${e}"
-  }
-}
-
-// --------------------------------------
-
-def updated() {
-  unsubscribe()
-  debug "Updated with settings: ${settings}"
-  def subscribed = [:]
-  selectedCapabilities.each{ capability ->
-    settings."${capability}Capability".each { thing ->
-      if (subscribed[thing.id]) {
-        return
-      }
-      subscribed[thing.id] = true
-      thing.supportedAttributes.each { attribute ->
-        debug "subscribe to attribute ${attribute.name}"
-        subscribe thing, attribute.name, eventForward
-      }
-      thing.supportedCommands.each { command ->
-        debug "subscribe to command ${command.name}"
-        subscribeToCommand thing, command.name, eventForward
-      }
-      debug "subscribed to thing ${thing.id}"
-    }
-  }
-  cleanUpTokens()
-}
-
-// --------------------------------------
-
-def cleanUpTokens() {
-
-  if (state.vendorToken) {
-    def params = [
-      uri: apiUrl() + "devices/${state.vendorUuid}/tokens/${state.vendorToken}",
-      headers: ["Authorization": "Bearer ${state.vendorBearerToken}"]
-    ]
-
-    debug "deleting url ${params.uri}"
-    try {
-      httpDelete(params) { response ->
-        debug "revoked token for ${state.vendorUuid}...?"
-      }
-    } catch (e) {
-      log.error "token delete error ${e}"
-    }
-  }
-
-  state.vendorBearerToken = null
-  state.vendorUuid = null
-  state.vendorToken = null
-
-  if (state.vendorOAuthToken) {
-    def params = [
-      uri: apiUrl() + "devices/${state.vendorOAuthUuid}",
-      headers: [
-        "meshblu_auth_uuid": state.vendorOAuthUuid,
-        "meshblu_auth_token": state.vendorOAuthToken
-      ]
-    ]
-
-    debug "deleting url ${params.uri}"
-    try {
-      httpDelete(params) { response ->
-        debug "deleted oauth device for ${state.vendorOAuthUuid}...?"
-      }
-    } catch (e) {
-      log.error "oauth token delete error ${e}"
-    }
-  }
-
-  state.vendorOAuthUuid = null
-  state.vendorOAuthToken = null
-
-}
-
-// --------------------------------------
-
-def getOauthCode() {
-  // revokeAccessToken()
-  // state.accessToken = createAccessToken()
-  debug "generated app access token ${state.accessToken}"
-
-  def postParams = [
-    uri: getVendorTokenPath(),
-    body: [
-      client_id: state.vendorOAuthUuid,
-      client_secret: state.vendorOAuthToken,
-      grant_type: "authorization_code",
-      code: params.code
-    ]
-  ]
-
-  def style = "<style type='text/css'>body{font-size:2em;padding:1em}</style>"
-  def startBody = "<html>${style}<body>"
-  def endBody = "</body></html>"
-  def goodResponse = "${startBody}<h1>Received Octoblu Token!</h1><h2>Press 'Done' to finish setup.</h2>${endBody}"
-  def badResponse = "${startBody}<h1>Something went wrong...</h1><h2>PANIC!</h2>${endBody}"
-  debug "authorizeToken with postParams ${postParams}"
-
-  try {
-    httpPost(postParams) { response ->
-      debug "response: ${response.data}"
-      state.vendorBearerToken = response.data.access_token
-      def bearer = new String((new Base64()).decode(state.vendorBearerToken)).split(":")
-      state.vendorUuid = bearer[0]
-      state.vendorToken = bearer[1]
-
-      debug "have octoblu tokens ${state.vendorBearerToken}"
-      render contentType: 'text/html', data: (state.vendorBearerToken ? goodResponse : badResponse)
-    }
-  } catch(e) {
-    log.error "second leg oauth error ${e}"
-    render contentType: 'text/html', data: badResponse
-  }
-}
-
-def getEventData(evt) {
-  return [
-    "date" : evt.date,
-    "id" : evt.id,
-    "data" : evt.data,
-    "description" : evt.description,
-    "descriptionText" : evt.descriptionText,
-    "displayName" : evt.displayName,
-    "deviceId" : evt.deviceId,
-    "hubId" : evt.hubId,
-    "installedSmartAppId" : evt.installedSmartAppId,
-    "isoDate" : evt.isoDate,
-    "isDigital" : evt.isDigital(),
-    "isPhysical" : evt.isPhysical(),
-    "isStateChange" : evt.isStateChange(),
-    "locationId" : evt.locationId,
-    "name" : evt.name,
-    "source" : evt.source,
-    "unit" : evt.unit,
-    "value" : evt.value,
-    "category" : "event",
-    "type" : "device:smart-thing"
-  ]
-}
-
-def eventForward(evt) {
-  def eventData = [ "devices" : [ "*" ], "payload" : getEventData(evt) ]
-
-  debug "sending event: ${groovy.json.JsonOutput.toJson(eventData)}"
-
-  def vendorDevice = state.vendorDevices[evt.deviceId]
-  if (!vendorDevice) {
-    log.error "aborting, vendor device for ${evt.deviceId} doesn't exist?"
-    return
-  }
-
-  debug "using device ${vendorDevice}"
-
-  def postParams = [
-    uri: apiUrl() + "messages",
-    headers: [
-      "meshblu_auth_uuid": vendorDevice.uuid,
-      "meshblu_auth_token": vendorDevice.token
-    ],
-    body: groovy.json.JsonOutput.toJson(eventData)
-  ]
-
-  try {
-    httpPostJson(postParams) { response ->
-      debug "sent off device event"
-    }
-  } catch (e) {
-    log.error "unable to send device event ${e}"
-  }
-}
-
-// --------------------------------------
-
-def postMessage() {
-  debug("received message data ${request.JSON}")
-  def foundDevice = false
-  selectedCapabilities.each{ capability ->
-    settings."${capability}Capability".each { thing ->
-      if (!foundDevice && thing.id == request.JSON.smartDeviceId) {
-        def vendorDevice = state.vendorDevices[thing.id]
-        foundDevice = true
-        if (vendorDevice.uuid == request.JSON.fromUuid) {
-          log.error "aborting message from self"
-          return
-        }
-
-        if (!request.JSON.command.startsWith("app-")) {
-          def args = []
-          if (request.JSON.args) {
-            request.JSON.args.each { k, v ->
-              args.push(v)
-            }
-          }
-
-          debug "command being sent: ${request.JSON.command}\targs to be sent: ${args}"
-          thing."${request.JSON.command}"(*args)
-        } else {
-          debug "calling internal command ${request.JSON.command}"
-          def commandData = [:]
-          switch (request.JSON.command) {
-            case "app-get-value":
-              debug "got command value"
-              thing.supportedAttributes.each { attribute ->
-                commandData[attribute.name] = thing.latestValue(attribute.name)
-              }
-              break
-            case "app-get-state":
-              debug "got command state"
-              thing.supportedAttributes.each { attribute ->
-                commandData[attribute.name] = thing.latestState(attribute.name)?.value
-              }
-              break
-            case "app-get-device":
-              debug "got command device"
-              commandData = [
-                "id" : thing.id,
-                "displayName" : thing.displayName,
-                "name" : thing.name,
-                "label" : thing.label,
-                "capabilities" : thing.capabilities.collect{ thingCapability -> return thingCapability.name },
-                "supportedAttributes" : thing.supportedAttributes.collect{ attribute -> return attribute.name },
-                "supportedCommands" : thing.supportedCommands.collect{ command -> return ["name" : command.name, "arguments" : command.arguments ] }
-              ]
-              break
-            case "app-get-events":
-              debug "got command events"
-              commandData.events = []
-              thing.events().each { event ->
-                commandData.events.push(getEventData(event))
-              }
-              break
-            default:
-              commandData.error = "unknown command"
-              debug "unknown command ${request.JSON.command}"
-          }
-          commandData.command = request.JSON.command
-          debug "with vendorDevice ${vendorDevice} for ${groovy.json.JsonOutput.toJson(commandData)}"
-
-          def postParams = [
-            uri: apiUrl() + "messages",
-            headers: ["meshblu_auth_uuid": vendorDevice.uuid, "meshblu_auth_token": vendorDevice.token],
-            body: groovy.json.JsonOutput.toJson([ "devices" : [ "*" ], "payload" : commandData ])
-          ]
-
-          debug "posting params ${postParams}"
-
-          try {
-            debug "calling httpPostJson!"
-            httpPostJson(postParams) { response ->
-              debug "sent off command result"
-            }
-          } catch (e) {
-            log.error "unable to send command result ${e}"
-          }
-
-        }
-      }
-    }
-  }
-}
-
-// --------------------------------------
-
-def postApp() {
-  debug("received app data ${request.JSON}")
-  if (state.vendorDevices[app.id].uuid == request.JSON.fromUuid) {
-    log.error "aborting message from self"
-    return
-  }
-  def args = []
-  if (request.JSON.args) {
-    request.JSON.args.each { k, v ->
-      args.push(v)
-    }
-  }
-
-  def commandData = [:]
-
-  switch (request.JSON.command) {
-    case "app-get-location":
-      debug "got command location"
-
-      def modes = []
-      location.modes.each { mode ->
-        modes.push([
-          "id" : mode.id,
-          "name" : mode.name
-        ])
-      }
-
-      def hubs = []
-      location.hubs.each { hub ->
-        debug "hub : ${hub}"
-        hubs.push([
-          "firmwareVersionString" : hub.firmwareVersionString,
-          "id" : hub.id,
-          "localIP" : hub.localIP,
-          "localSrvPortTCP" : hub.localSrvPortTCP,
-          "name" : hub.name,
-          "type" : hub.type,
-          "zigbeeEui" : hub.zigbeeEui,
-          "zigbeeId" : hub.zigbeeId
-        ])
-      }
-
-      commandData = [
-        "contactBookEnabled" : location.contactBookEnabled,
-        "id" : location.id,
-        "latitude" : location.latitude,
-        "longitude" : location.longitude,
-        "temperatureScale" : location.temperatureScale,
-        "timeZone" : location.timeZone.getID(),
-        "zipCode" : location.zipCode,
-        "mode" : location.mode,
-        "modes" : modes,
-        "hubs" : hubs
-      ]
-
-      debug "copied location!"
-      debug commandData
-      break
-
-    case "app-get-devices":
-      debug "got command devices"
-      commandData.devices = state.vendorDevices.collect { k, v -> [ "smartDeviceId" : k, "uuid" : v.uuid ] }
-      break
-
-    case "app-set-mode":
-      location.setMode(*args)
-      commandData.mode = args[0]
-      break
-
-    default:
-      commandData.error = "unknown command"
-      debug "unknown command ${request.JSON.command}"
-  }
-  commandData.command = request.JSON.command
-  debug "sending ${commandData}"
-
-  def vendorDevice = state.vendorDevices[app.id]
-  debug "with vendorDevice ${vendorDevice} for ${groovy.json.JsonOutput.toJson(commandData)}"
-
-  def postParams = [
-    uri: apiUrl() + "messages",
-    headers: [ "meshblu_auth_uuid" : vendorDevice.uuid, "meshblu_auth_token" : vendorDevice.token ],
-    body: groovy.json.JsonOutput.toJson([ "devices" : [ "*" ], "payload" : commandData ])
-  ]
-
-  debug "posting params ${postParams}"
-
-  try {
-    debug "calling httpPostJson!"
-    httpPostJson(postParams) { response ->
-      debug "sent off command result"
-    }
-  } catch (e) {
-    log.error "unable to send command result ${e}"
-  }
-}
-
-// --------------------------------------
-
-private debug(logStr) {
-  if (USE_DEBUG)
-    log.debug logStr
-}
-
-String toQueryString(Map m) {
-  return m.collect { k, v -> "${k}=${URLEncoder.encode(v.toString())}" }.sort().join("&")
-}
-
-def initialize()
-{
-  debug "Initialized with settings: ${settings}"
-}
-
-def uninstalled()
-{
-  debug "In uninstalled ${state.removeDevices}"
-  if (state.removeDevices) {
-    state.vendorDevices.each  { k, device ->
-      def params = [
-        uri: apiUrl() + "devices/${device.uuid}",
-        headers: [ "meshblu_auth_uuid" : device.uuid, "meshblu_auth_token" : device.token ],
-      ]
-
-      debug "deleting url ${params.uri}"
-      try {
-        httpDelete(params) { response ->
-          debug "delete device ${device.uuid}"
-        }
-      } catch (e) {
-        log.error "token delete error ${e}"
-      }
-    }
-  }
-}
-
-def installed() {
-  debug "Installed with settings: ${settings}"
-  state.installed = true
-}
-
-private Boolean canInstallLabs()
-{
-  return hasAllHubsOver("000.011.00603")
-}
-
-private Boolean hasAllHubsOver(String desiredFirmware)
-{
-  return realHubFirmwareVersions.every { fw -> fw >= desiredFirmware }
-}
-
-private List getRealHubFirmwareVersions()
-{
-  return location.hubs*.firmwareVersionString.findAll { it }
-}

smartapps/juano2310/jawbone-button-notifier.src/jawbone-button-notifier.groovy

@@ -62,7 +62,7 @@ def initialize() {
 }
 
 def sendit(evt) {
-	log.debug "$evt.value: $evt, $settings"
+	log.debug "$evt.value: $evt"
 	sendMessage()
 }
 
@@ -80,6 +80,6 @@ def sendMessage() {
         sendSms phone3, msg
     }
     if (!phone1 && !phone2 && !phone3) {
-    	sendPush msg    
+    	sendPush msg
     }
 }

smartapps/smartthings/hue-connect.src/hue-connect.groovy

@@ -133,14 +133,23 @@ def bulbDiscovery() {
 	state.inBulbDiscovery = true
 	def bridge = null
 	if (selectedHue) {
-        bridge = getChildDevice(selectedHue)
-        subscribe(bridge, "bulbList", bulbListData)
+		bridge = getChildDevice(selectedHue)
+		subscribe(bridge, "bulbList", bulbListData)
 	}
-    state.bridgeRefreshCount = 0
-	def bulboptions = bulbsDiscovered() ?: [:]
-	def numFound = bulboptions.size() ?: 0
-    if (numFound == 0)
-    	app.updateSetting("selectedBulbs", "")
+	state.bridgeRefreshCount = 0
+	def allLightsFound = bulbsDiscovered() ?: [:]
+
+	// List lights currently not added to the user (editable)
+	def newLights = allLightsFound.findAll {getChildDevice(it.key) == null} ?: [:]
+	newLights = newLights.sort {it.value.toLowerCase()}
+
+	// List lights already added to the user (not editable)
+	def existingLights = allLightsFound.findAll {getChildDevice(it.key) != null} ?: [:]
+	existingLights = existingLights.sort {it.value.toLowerCase()}
+
+	def numFound = newLights.size() ?: 0
+	if (numFound == 0)
+		app.updateSetting("selectedBulbs", "")
 
 	if((bulbRefreshCount % 5) == 0) {
 		discoverHueBulbs()
@@ -148,14 +157,25 @@ def bulbDiscovery() {
 	def selectedBridge = state.bridges.find { key, value -> value?.serialNumber?.equalsIgnoreCase(selectedHue) }
 	def title = selectedBridge?.value?.name ?: "Find bridges"
 
+	// List of all lights previously added shown to user
+	def existingLightsDescription = ""
+	if (existingLights) {
+		existingLights.each {
+			if (existingLightsDescription.isEmpty()) {
+				existingLightsDescription += it.value
+			} else {
+				 existingLightsDescription += ", ${it.value}"
+			}
+		}
+	}
 
 	return dynamicPage(name:"bulbDiscovery", title:"Light Discovery Started!", nextPage:"", refreshInterval:refreshInterval, install:true, uninstall: true) {
 		section("Please wait while we discover your Hue Lights. Discovery can take five minutes or more, so sit back and relax! Select your device below once discovered.") {
-			input "selectedBulbs", "enum", required:false, title:"Select Hue Lights (${numFound} found)", multiple:true, options:bulboptions
+			input "selectedBulbs", "enum", required:false, title:"Select Hue Lights to add (${numFound} found)", multiple:true, options:newLights
+			paragraph title: "Previously added Hue Lights (${existingLights.size()} added)", existingLightsDescription
 		}
 		section {
 			href "bridgeDiscovery", title: title, description: "", state: selectedHue ? "complete" : "incomplete", params: [override: true]
-
 		}
 	}
 }
@@ -290,8 +310,11 @@ def manualRefresh() {
 }
 
 def uninstalled(){
+	// Remove bridgedevice connection to allow uninstall of smartapp even though bridge is listed
+	// as user of smartapp
+	app.updateSetting("bridgeDevice", null)
 	state.bridges = [:]
-    state.username = null
+	state.username = null
 }
 
 // Handles events to add new bulbs
@@ -415,23 +438,32 @@ def addBridge() {
 				// Hue uses last 6 digits of MAC address as ID number, this number is shown on the bottom of the bridge
 				def idNumber = getBridgeIdNumber(selectedHue)
 				d = addChildDevice("smartthings", "Hue Bridge", selectedHue, vbridge.value.hub, ["label": "Hue Bridge ($idNumber)"])
-				d?.completedSetup = true
- 				log.debug "created ${d.displayName} with id ${d.deviceNetworkId}"
-                def childDevice = getChildDevice(d.deviceNetworkId)
-				updateBridgeStatus(childDevice)
-				childDevice.sendEvent(name: "idNumber", value: idNumber)
- 				if (vbridge.value.ip && vbridge.value.port) {
-                	if (vbridge.value.ip.contains(".")) {
-                    	childDevice.sendEvent(name: "networkAddress", value: vbridge.value.ip + ":" +  vbridge.value.port)
-                        childDevice.updateDataValue("networkAddress", vbridge.value.ip + ":" +  vbridge.value.port)
-                    } else {
-                    	childDevice.sendEvent(name: "networkAddress", value: convertHexToIP(vbridge.value.ip) + ":" +  convertHexToInt(vbridge.value.port))
-                        childDevice.updateDataValue("networkAddress", convertHexToIP(vbridge.value.ip) + ":" +  convertHexToInt(vbridge.value.port))
-                    }
+				if (d) {
+					// Associate smartapp to bridge so user will be warned if trying to delete bridge
+					app.updateSetting("bridgeDevice", [type: "device.hueBridge", value: d.id])
+
+					d.completedSetup = true
+					log.debug "created ${d.displayName} with id ${d.deviceNetworkId}"
+					def childDevice = getChildDevice(d.deviceNetworkId)
+					updateBridgeStatus(childDevice)
+					childDevice.sendEvent(name: "idNumber", value: idNumber)
+
+
+					if (vbridge.value.ip && vbridge.value.port) {
+						if (vbridge.value.ip.contains(".")) {
+							childDevice.sendEvent(name: "networkAddress", value: vbridge.value.ip + ":" +  vbridge.value.port)
+							childDevice.updateDataValue("networkAddress", vbridge.value.ip + ":" +  vbridge.value.port)
+						} else {
+							childDevice.sendEvent(name: "networkAddress", value: convertHexToIP(vbridge.value.ip) + ":" +  convertHexToInt(vbridge.value.port))
+							childDevice.updateDataValue("networkAddress", convertHexToIP(vbridge.value.ip) + ":" +  convertHexToInt(vbridge.value.port))
+						}
+					} else {
+						childDevice.sendEvent(name: "networkAddress", value: convertHexToIP(vbridge.value.networkAddress) + ":" +  convertHexToInt(vbridge.value.deviceAddress))
+						childDevice.updateDataValue("networkAddress", convertHexToIP(vbridge.value.networkAddress) + ":" +  convertHexToInt(vbridge.value.deviceAddress))
+					}
 				} else {
-					childDevice.sendEvent(name: "networkAddress", value: convertHexToIP(vbridge.value.networkAddress) + ":" +  convertHexToInt(vbridge.value.deviceAddress))
-                    childDevice.updateDataValue("networkAddress", convertHexToIP(vbridge.value.networkAddress) + ":" +  convertHexToInt(vbridge.value.deviceAddress))
-                }
+					log.error "Failed to create Hue Bridge device"
+				}
 			}
 		} else {
 			log.debug "found ${d.displayName} with id $selectedHue already exists"
@@ -726,6 +758,10 @@ def isValidSource(macAddress) {
 	return (vbridges?.find {"${it.value.mac}" == macAddress}) != null
 }
 
+def isInBulbDiscovery() {
+	return state.inBulbDiscovery
+}
+
 /////////////////////////////////////
 //CHILD DEVICE METHODS
 /////////////////////////////////////
@@ -1017,12 +1053,12 @@ def setColor(childDevice, huesettings) {
     log.debug "Executing 'setColor($huesettings)'"
 
     updateInProgress()
-    
+
     def value = [:]
     def hue = null
     def sat = null
     def xy = null
-    
+
 	// For now ignore model to get a consistent color if same color is set across multiple devices
 	// def model = state.bulbs[getId(childDevice)]?.modelid
     if (huesettings.hex != null) {
@@ -1038,11 +1074,12 @@ def setColor(childDevice, huesettings) {
 	else
 		value.hue = Math.min(Math.round(childDevice.device?.currentValue("hue") * 65535 / 100), 65535)
 
-        if (huesettings.saturation != null)   
+        if (huesettings.saturation != null)
 		value.sat = Math.min(Math.round(huesettings.saturation * 254 / 100), 254)
 	else
 		value.sat = Math.min(Math.round(childDevice.device?.currentValue("saturation") * 254 / 100), 254)
 
+/* Disabled for now due to bad behavior via Lightning Wizard
 	if (!value.xy) {
 		// Below will translate values to hex->XY to take into account the color support of the different hue types
 		def hex = colorUtil.hslToHex((int) huesettings.hue, (int) huesettings.saturation)
@@ -1050,6 +1087,7 @@ def setColor(childDevice, huesettings) {
 		// Once groups, or scenes are introduced it might be a good idea to use unique models again
 		value.xy = calculateXY(hex)
     }
+*/
 
     // Default behavior is to turn light on
     value.on = true
@@ -1059,7 +1097,7 @@ def setColor(childDevice, huesettings) {
             value.on = false
         else if (huesettings.level == 1)
             value.bri = 1
-        else 
+        else
 			value.bri = Math.min(Math.round(huesettings.level * 254 / 100), 254)
     }
     value.alert = huesettings.alert ? huesettings.alert : "none"

smartapps/smartthings/life360-connect.src/life360-connect.groovy

@@ -74,8 +74,6 @@ def authPage()
 
 		def redirectUrl = oauthInitUrl()
     
-  	  	log.debug "RedirectURL = ${redirectUrl}"
-
 		return dynamicPage(name: "Credentials", title: "Life360", nextPage:"listCirclesPage", uninstall: uninstallOption, install:false) {
 		    section {
     			href url:redirectUrl, style:"embedded", required:false, title:"Life360", description:description
@@ -257,8 +255,6 @@ def initializeLife360Connection() {
 	def oauthClientId = appSettings.clientId
 	def oauthClientSecret = appSettings.clientSecret
 
-	log.debug "Installed with settings: ${settings}"
-
 	initialize()
     
     def username = settings.username
@@ -269,8 +265,6 @@ def initializeLife360Connection() {
   	def basicCredentials = "${oauthClientId}:${oauthClientSecret}"
     def encodedCredentials = basicCredentials.encodeAsBase64().toString()
     
-    log.debug "Encoded Creds: ${encodedCredentials}"
-
     
     // call life360, get OAUTH token using password flow, save
     // curl -X POST -H "Authorization: Basic cFJFcXVnYWJSZXRyZTRFc3RldGhlcnVmcmVQdW1hbUV4dWNyRUh1YzptM2ZydXBSZXRSZXN3ZXJFQ2hBUHJFOTZxYWtFZHI0Vg==" 
@@ -284,8 +278,6 @@ def initializeLife360Connection() {
     				"username=${username}&"+
                     "password=${password}"
 
-    log.debug "Post Body: ${postBody}"
-	   
     def result = null
     
     try {
@@ -295,7 +287,6 @@ def initializeLife360Connection() {
 		}
         if (result.data.access_token) {
        		state.life360AccessToken = result.data.access_token
-       		log.debug "Access Token = ${state.life360AccessToken}"
             return true;
    		}
 		log.debug "Response=${result.data}"
@@ -533,8 +524,6 @@ def createCircleSubscription() {
         
     def postBody =  "url=${hookUrl}"
 
-    log.debug "Post Body: ${postBody}"
-	   
     def result = null
     
     try {
@@ -586,8 +575,6 @@ def updated() {
         
         	// log.debug "After Find Attempt."
 
-       		log.debug "Member Id = ${member.id}, Name = ${member.firstName} ${member.lastName}, Email Address = ${member.loginEmail}"
-        
         	// log.debug "External Id=${app.id}:${member.id}"
        
        		// create the device

smartapps/smartthings/notify-me-when.src/notify-me-when.groovy

@@ -48,9 +48,9 @@ preferences {
 	}
 	section("Via a push notification and/or an SMS message"){
 		input("recipients", "contact", title: "Send notifications to") {
-			input "phone", "phone", title: "Phone Number (for SMS, optional)", required: false
+			input "phone", "phone", title: "Enter a phone number to get SMS", required: false
 			paragraph "If outside the US please make sure to enter the proper country code"
-			input "pushAndPhone", "enum", title: "Both Push and SMS?", required: false, options: ["Yes", "No"]
+			input "pushAndPhone", "enum", title: "Notify me via Push Notification", required: false, options: ["Yes", "No"]
 		}
 	}
 	section("Minimum time between messages (optional, defaults to every message)") {
@@ -111,19 +111,24 @@ private sendMessage(evt) {
 	if (location.contactBookEnabled) {
 		sendNotificationToContacts(msg, recipients, options)
 	} else {
-		if (!phone || pushAndPhone != 'No') {
-			log.debug 'sending push'
-			options.method = 'push'
-			//sendPush(msg)
-		}
 		if (phone) {
 			options.phone = phone
-			log.debug 'sending SMS'
-			//sendSms(phone, msg)
+			if (pushAndPhone != 'No') {
+				log.debug 'Sending push and SMS'
+				options.method = 'both'
+			} else {
+				log.debug 'Sending SMS'
+				options.method = 'phone'
+			}
+		} else if (pushAndPhone != 'No') {
+			log.debug 'Sending push'
+			options.method = 'push'
+		} else {
+			log.debug 'Sending nothing'
+			options.method = 'none'
 		}
 		sendNotification(msg, options)
 	}
-
 	if (frequency) {
 		state[evt.deviceId] = now()
 	}