dbarentine / udi-totalconnect-poly

UDI Polyglot v2 Node Server for Total Connect 2 security systems
MIT License
1 stars 0 forks source link

Feature Request #4

Closed craigwitter closed 4 years ago

craigwitter commented 5 years ago

Could disarming also be added? I would assume a property would need to be added for user code to do the disarm.

Thanks in advance!

dbarentine commented 4 years ago

@craigwitter Sorry about the delay on this. I haven't forgotten about this I just haven't had a chance to get back to it yet. But I think I'll be able to knock it out before the end of the year.

dbarentine commented 4 years ago

@craigwitter Sorry this took so long for me to get to. I've added it in version 1.2.0 which should be showing up in Polyglot shortly.

By default the ability to disarm is disabled. You just need to update allow_disarming to true to enable it.

craigwitter commented 4 years ago

Thank you. Should this work on Polyisy?

Craig

On Wed, Feb 5, 2020 at 2:20 AM Dane Barentine notifications@github.com wrote:

Closed #4 https://github.com/dbarentine/udi-totalconnect-poly/issues/4.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/dbarentine/udi-totalconnect-poly/issues/4?email_source=notifications&email_token=ADDNYG6R75JJC6ERV54V46DRBJSDRA5CNFSM4IF2IITKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWNL5ODA#event-3008878348, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADDNYG4NBT7A5GHPTGGOCDLRBJSDRANCNFSM4IF2IITA .

dbarentine commented 4 years ago

There are some dependency issues on Polisy at the moment. See Issue #5.

I've been working with UDI on it and think we are close to having a resolution. Once the dependency issue is resolved it works without issues. I'll update the other issue once we've got a fix.