Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Opener: Expose "Continous Mode" and "Ring To Open" as Switch instead of Lock #50

Closed
flobernd opened this issue Oct 16, 2019 · 5 comments

Comments

@flobernd
Copy link

Hi there,

First of all: Nice plugin!

Could you please expose the additional "Continous Mode" and "Ring To Open" locks as a Switch service instead of Locks or make this an configurable option?

Best regards

@benzman81
Copy link
Owner

I decided to use locks as this is a security issue. What is the reason or use case for using switches instead?

@flobernd
Copy link
Author

flobernd commented Oct 17, 2019

It's just a personal preference I guess. In my opinon having just one single Lock accessory for each physical lock is way more intuitive as it represents the reality with a one to one mapping of real device to HomeKit device.

If you are concerned about security in terms that the user does not always sees an open lock icon, you could probably just always show the "main" lock as "opened" when "Continous Mode" or "Ring To Open" is active (like you do it for "Ring To Open" when "Continous Mode" is enabled). As you can't manually close a door controlled by the Nuki Opener anyways, it won't be a problem if the lock stays open in HomeKit until you disable both options.

Anyways, I can understand if you decide to not support multiple configurations.

@benzman81
Copy link
Owner

Its not about something in the UI. If anyone has your smartphone, he can activate all switches using siri without unlocking the phone. This way he could activate ringtoopen or cm and get into the house...

@flobernd
Copy link
Author

Oh well, I did not know that. Would you mind showing the "main" lock as "closed", while one of the options is active? I think that could be a good addition anyways.

@benzman81
Copy link
Owner

Sorry for the delay. We have no influence on what the „main“ lock looks like. If one of the child locks is open, then the „main“ lock is shown open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants