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

[nuki] Support for SmartLock 3.0 and SmartDoor #12005

Merged
merged 1 commit into from
Jan 10, 2022
Merged

Conversation

janvyb
Copy link
Contributor

@janvyb janvyb commented Jan 9, 2022

This PR adds support for recently released products - Smart Lock 3.0 and Smart Door.

This is pretty simple PR since both new devices behave the same as Smart Lock 2.0, the only change needed is to allow devices with new deviceTypes (3, 4) to be recognized by discovery as smartlock.

Added new deviceType read only property (since now there are 3 smartlock deviceTypes) on smartlock thing because all actions on bridge require deviceType to be sent with request. This is just to be sure, there does not seem to be any difference sending deviceType 0, 3 or 4 (my smartlock 2.0 works the same with any of these values), this property seems to only exist to distinguish smartlock and opener calls.

I also updated documentation on smartlockDoorState channel to include new values introduced in v 1.13.1 of the bridge API.

@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/new-openhab2-binding-for-nuki-smart-locks/25940/177

@wborn wborn added the enhancement An enhancement or new feature for an existing add-on label Jan 9, 2022
Signed-off-by: Jan Vybíral <jan.vybiral1@gmail.com>
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lolodomo lolodomo merged commit dc0a974 into openhab:main Jan 10, 2022
@lolodomo lolodomo added this to the 3.3 milestone Jan 10, 2022
moesterheld pushed a commit to moesterheld/openhab-addons that referenced this pull request Jan 18, 2022
Signed-off-by: Jan Vybíral <jan.vybiral1@gmail.com>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Jan 28, 2022
Signed-off-by: Jan Vybíral <jan.vybiral1@gmail.com>
volkmarnissen pushed a commit to volkmarnissen/openhab-addons that referenced this pull request Mar 3, 2022
Signed-off-by: Jan Vybíral <jan.vybiral1@gmail.com>
NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this pull request Apr 27, 2022
Signed-off-by: Jan Vybíral <jan.vybiral1@gmail.com>
Signed-off-by: Nick Waterton <n.waterton@outlook.com>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
Signed-off-by: Jan Vybíral <jan.vybiral1@gmail.com>
andrasU pushed a commit to andrasU/openhab-addons that referenced this pull request Nov 12, 2022
Signed-off-by: Jan Vybíral <jan.vybiral1@gmail.com>
Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants