judge / homebridge-hikconnect

A Homebridge plugin to communicate with Hikvision smart doorbells via Hik-Connect cloud and allows lock to be unlocked. It exposes doorbell locks as a lock accessories to Homekit.
MIT License
18 stars 1 forks source link

One Lock showing out of two #2

Closed kapps7 closed 2 years ago

kapps7 commented 2 years ago

Hey I thought this is a better place to present the issue.

I have one lock of the two working but more showing from my gen 2 intercom as well as multiple other locks which seem to match the number of cameras I have in my nvr ie 9.

Wondering what I need to do to make the second lock work. Thanks

Screen Shot 2021-11-08 at 10 59 37 am

s

judge commented 2 years ago

Hi, That is strange, it seems these devices have lock somehow. Can you help me debugging this?

kapps7 commented 2 years ago

Hey thanks for replying. I have the output you have asked for. Where is the dm option ;) Sent the link via reddit.

judge commented 2 years ago

Thanks! It seems Github removed this feature a while ago. :( Anyway, I got the response, I'll have a look soon.

judge commented 2 years ago

Hi! I just published a new version which should solve your issue (at least I hope... sorry it took so long, I had to do some major refactoring in the code to support multiple locks in the same lock channel (and also there is no documentation for this API)) Could you please check it? Also it's a good idea to clear your devices caches first from the UI: Homebridge Settings / Remove Single Cached Accessory / search for your lock(s)

kapps7 commented 2 years ago

I cleared the cached accessories and now I only see two. I am just waiting for my client to return so I can test it properly. Thank you very much for putting in the work, this will be very useful to Hik owners. I have a suggestion: Most intercom owners who are using both the lock relays are most likely using one of them for a garage or gate. It would be useful as a setting to show one of the locks as a garage.

As you know the garage accessory is treated differently in Homekit. I can use a dummy garage switch but it would be a cool feature.

I will get back to you soon after testing. Thanks Judge!

judge commented 2 years ago

Actually I don't know what is the difference (in Homekit point of view) between garage and locks. Do you think these would be better as garage doors? The way how it works now is kind of hacky as well because (at least in my case) the "lock" works as a latch. So whenever I unlock the door it will be unlocked for a certain amount of time (5-10 seconds) and then it automatically locks itself, so I had to manually set the status of the lock to locked after few seconds.

kapps7 commented 2 years ago

Hey mate, the early signs are good. Both locks are working. I am testing the auto lock 'hack' you are referring to see how ours performs. Ivms allows you to set that in the app so I guess thats an option for you down the track to add a setting to the plugin to allow uses to define the period for lock status.

Garage and locks are both secure accessories so they abide homekit rules for this category which include things like the owners device needs to be on the network and for an approval request via the phone if enabled. Garage however also has an obstruction sensor and has different features eg. via CarPlay. So if you are driving home with CarPlay and you have a garage accessory, it will automatically pop up when you approach home on your display to tap. This is not a user automation, just built into homekit

So in answer to your question I think a user option to set either of the locks as a garage would be great. That way the user customises it however their intercom is setup

judge commented 2 years ago

Thanks for the info! Regarding the timeout, is it possible to set it by lock? So is it possible to set 5 seconds to one lock and 15 seconds to an another? Choosing between Garage and Lock is a good idea but the problem is that Homebridge registers the devices dynamically and you have to decide what type of accessory you want to register in advance. I have no idea how can I make it configurable.

judge commented 2 years ago

I'll close this for now, let me know if locks are still not working for you!