SmartThingsCommunity / SmartThingsPublic

SmartThings open-source DeviceType Handlers and SmartApps code
https://developer-preview.smartthings.com/docs/devices/hub-connected/get-started
2.56k stars 89.88k forks source link

SmartThings MultiSensor 6 #1069

Closed jvmahon closed 8 years ago

jvmahon commented 8 years ago

The published device handler for the Aeon Labs Multisensor 6 has a serious error which prevents proper installation of the Multisensor 6. Specifically, the device handler fingerprint is incorrect.

As published, the fingerprint lists an outClusters "0x5A", but the "0x5A" needs to be part of the inclusters, and there should be no outClusters for this device.

Fingerprint should read:

fingerprint deviceId: "0x2101", inClusters: "0x5E,0x86,0x72,0x59,0x85,0x73,0x71,0x84,0x80,0x30,0x31,0x70,0x7A,0x5A"

workingmonk commented 8 years ago

@dkirker @tylerlange @mckeed

tylerlange commented 8 years ago

@dkirker i think you were working on this, correct?

dkirker commented 8 years ago

@tylerlange @workingmonk @mckeed Yup! I am working on a PR to fix this and another bug. :) Should come today or tomorrow.

jvmahon commented 8 years ago

Thanks!

From: Donald C. Kirker [mailto:notifications@github.com] Sent: Friday, July 22, 2016 7:53 PM To: SmartThingsCommunity/SmartThingsPublic SmartThingsPublic@noreply.github.com Cc: jvmahon jvm33@columbia.edu; Author author@noreply.github.com Subject: Re: [SmartThingsCommunity/SmartThingsPublic] SmartThings MultiSensor 6 (#1069)

@tylerlange https://github.com/tylerlange @workingmonk https://github.com/workingmonk @mckeed https://github.com/mckeed Yup! I am working on a PR to fix this and another bug. :) Should come today or tomorrow.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/SmartThingsCommunity/SmartThingsPublic/issues/1069#issuecomment-234683749 , or mute the thread https://github.com/notifications/unsubscribe-auth/AOXTtvR-Q6p5Nog4_Ofp_jSNPivkqMZ7ks5qYVfcgaJpZM4JSENt .

dkirker commented 8 years ago

Expected to be deployed on 8/2-8/3.

workingmonk commented 8 years ago

closing the issue as the code has been deployed