WebThingsIO / zwave-adapter

Z-Wave adapter add-on for WebThings Gateway
Mozilla Public License 2.0
15 stars 20 forks source link

kind of partial door sensor support #60

Closed cmatheson closed 5 years ago

cmatheson commented 5 years ago

Ecolink door sensors are being classified as "Sensor Notification" which wasn't being handled at all. This change handles "Sensor Notification" objects the same way as "Sensor Binary" objects. The end result is that there is an active/inactive value reported from the door sensor to the gateway interface.

I would really prefer to have proper door sensor support in the gateway UI, so any tips on getting there would be appreciated (I'm assuming no zwave door sensors are supported currently, since the @type is never classified as "DoorSensor").

dhylands commented 5 years ago

I've tested with an everspring door/window sensor. There doesn't appear to be anything in zwave to tell the adapter that its a door sensor.

The UI allows a thing to support multiple types, so I think that a reasonable compromise would be to allow a binary sensor to also be a door sensor and then the user can choose when they add the device.