search

QuailAutomation / openhab2-addons

The next-generation open Home Automation Bus (openHAB)
Eclipse Public License 2.0
8 stars 7 forks source link

Omnilink Lock Support #63

Closed swamiller closed 5 years ago

swamiller commented 7 years ago

Add the ability to control door locks (Access Control in HAI/PC Access terms).

boc-tothefuture commented 7 years ago

Looked at the API. This looks like it is possible.

boc-tothefuture commented 5 years ago

@swamiller - do you still want this?

swamiller commented 5 years ago

Yes, please. I would still prefer controlling/status my locks through my alarm panel (as master) versus OH. I have some creative flag use now to make it work in OH.

boc-tothefuture commented 5 years ago

@swamiller - I have added lock support, but I have no way to verify it works. Can you install this test version and provide feedsback?

https://github.com/boc-tothefuture/openhab2-addons/releases/download/locks/org.openhab.binding.omnilink-2.4.0-SNAPSHOT.jar

I will submit a PR once we know it functions.

swamiller commented 5 years ago

@boc-tothefuture So far, so good! I installed the test version and created new door lock items. They seem to function ok for the first install. I'll do more testing and update.

boc-tothefuture commented 5 years ago

Ok - that is good considering I could do very very limited testing. :-)

swamiller commented 5 years ago

I removed all my work around flags from my Omni, created OH items for each lock, and all seems to function as desired.

One question, there is a logEvent for "LogEvent changed from DOOR LOCKED to DOOR UNLOCKED" which is helpful. Is there a way to append the item name to know which door? Not a deal breaker, more of a question.

Thank you!!

boc-tothefuture commented 5 years ago

@swamiller - If I am understanding you correctly, you are talking about the logEvent text which comes from polling the omni. The Omni is the one responsible for that text, the binding doesn't modify it.

swamiller commented 5 years ago

LGTM, thank you @boc-tothefuture!