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

Bridge API: Bell Ring of Nuki Opener #40

Closed
ooooli opened this issue Dec 22, 2019 · 18 comments
Closed

Bridge API: Bell Ring of Nuki Opener #40

ooooli opened this issue Dec 22, 2019 · 18 comments
Assignees
Labels
enhancement New feature or request foreign issue Issue of a foreign repository verify-fix Verify if the implemented fix has solved the issue

Comments

@ooooli
Copy link
Contributor

ooooli commented Dec 22, 2019

Currently, the states are only reflecting the Lock functions.
e.g. ringing (state #224), recognized by the Opener is not announced.
"224": "Ringing",

{"id":"XXXXXXXXXXXX","smartlockId":XXXXXXXXXX,"deviceType":2,"name":"","action":224,"trigger":0,"state":0,"autoUnlock":false,"date":"2019-12-22T15:00:36.000Z","openerLog":{"activeCm":false,"activeRto":false,"source":0,"flagGeoFence":false,"flagForce":false,"flagDoorbellSuppression":false}}

@Zefau Zefau self-assigned this Jan 2, 2020
@Zefau Zefau added the bug Something isn't working label Jan 2, 2020
@Zefau
Copy link
Collaborator

Zefau commented Jan 2, 2020

I will have a look at this shortly. Thanks for the report.

@Zefau
Copy link
Collaborator

Zefau commented Feb 3, 2020

What's your source regarding state #224 being Ringing? The ioBroker log? Is the log entry you have provided generated by the nuki-extended adapter?

According to the API documentation (both Bridge API and Web API), state 224 does actually not exist.

@Zefau
Copy link
Collaborator

Zefau commented Feb 3, 2020

Apparently, the state is delivered with the log entries from the Nuki Opener (according to the Web API documentation).

Though, if that state is not delivered by the Bridge API callback there is no possibility to announce this via the nuki-extended adapter.

I will open an issue at the Nuki developer forum and confirm.

@Zefau
Copy link
Collaborator

Zefau commented Feb 3, 2020

@Zefau
Copy link
Collaborator

Zefau commented Feb 4, 2020

@ooooli please sign up in the Nuki developer forum and vote for the related feature request, see https://developer.nuki.io/t/nuki-bridge-api-bell-ring-of-nuki-opener-as-trigger-for-callbacks/2557

The ringing event is not delivered by Nuki as of now and that feature request asks for its implementation.

@ooooli
Copy link
Contributor Author

ooooli commented Feb 4, 2020

@ooooli please sign up in the Nuki developer forum and vote for the related feature request, see https://developer.nuki.io/t/nuki-bridge-api-bell-ring-of-nuki-opener-as-trigger-for-callbacks/2557

The ringing event is not delivered by Nuki as of now and that feature request asks for its implementation.

Done, thanks for following up

@Zefau Zefau added the foreign issue Issue of a foreign repository label Feb 9, 2020
@Zefau Zefau changed the title Please add Opener States Nuki API: Bell Ring of Nuki Opener May 22, 2020
@Zefau Zefau added enhancement New feature or request and removed bug Something isn't working labels May 22, 2020
@nachtschatt3n
Copy link

do we have ideas without the API supporting it?

@Zefau
Copy link
Collaborator

Zefau commented Jun 23, 2020

How is that supposed to work? Any information is being retrieved from the Bridge or Web API.

@nachtschatt3n
Copy link

well right, my ideas so far :

  • web scrape it from the nuki website (15-sec polling) (they should feel the pain)
  • try to catch the request going to the nuki servers via proxy (let's hope they are not doing SSL pinning)
  • catch the notification on the app (android is probably easier)
  • an Arduino with a microphone listening for the doorbell ?! calling iobroker URLs ?!

@ooooli
Copy link
Contributor Author

ooooli commented Aug 10, 2020

Nuki added a new state "ringactionState" into theit beta firmware:
https://developer.nuki.io/t/bridge-beta-fw-2-7-0-1-17-0/6792
If this is accessible via adapter, the issue may be solved. I have installed the beta on my Nuki Bridge and Opener and available for testing the functionality.

Edit: the state is available, after restarting the adapter and notifying the Doorbell correctly. So this issue can be closed, once the beta feature is public. The description in the adapter only needs to be adopted (from "(no description given)".

@Zefau
Copy link
Collaborator

Zefau commented Aug 10, 2020

My bridge is currently sent in for maintenance. Could you screenshot the states without description? I will add them then. Thanks

@Zefau Zefau changed the title Nuki API: Bell Ring of Nuki Opener Bridge API: Bell Ring of Nuki Opener Aug 10, 2020
@Zefau
Copy link
Collaborator

Zefau commented Aug 10, 2020

@ooooli Do you own a lock as well and could verify, if the door sensor (see #71) is working correctly with the fw 1.6 / 1.7 as well?

@ooooli
Copy link
Contributor Author

ooooli commented Aug 10, 2020

My bridge is currently sent in for maintenance. Could you screenshot the states without description? I will add them then. Thanks
Sure. The new states are created as following:

Bildschirmfoto 2020-08-10 um 15 32 50

@ooooli
Copy link
Contributor Author

ooooli commented Aug 10, 2020

@ooooli Do you own a lock as well and could verify, if the door sensor (see #71) is working correctly with the fw 1.6 / 1.7 as well?

Unfortunately, I don't have a lock. Only the bridge & Opener setup.

@Zefau
Copy link
Collaborator

Zefau commented Aug 10, 2020

@ooooli @nachtschatt3n with the latest adapter version 2.3.0 (from Github) the states state.ringState and state.ringStateUpdate should be available and reflect the bell ring. Could you please test and verify?

@Zefau Zefau added the verify-fix Verify if the implemented fix has solved the issue label Aug 10, 2020
@nachtschatt3n
Copy link

@Zefau sure I can check it tomorrow but don't I need the beta firmware for this?

@ooooli
Copy link
Contributor Author

ooooli commented Aug 11, 2020

@ooooli @nachtschatt3n with the latest adapter version 2.3.0 (from Github) the states state.ringState and state.ringStateUpdate should be available and reflect the bell ring. Could you please test and verify?

The states are now named correctly:
Bildschirmfoto 2020-08-11 um 08 00 10

Cross fingers, that Nuki soon adds this to the standard firmware.

@Zefau
Copy link
Collaborator

Zefau commented Aug 11, 2020

@Zefau sure I can check it tomorrow but don't I need the beta firmware for this?

@nachtschatt3n You do. It's only possible with the beta firmware for now.

@Zefau Zefau closed this as completed Aug 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request foreign issue Issue of a foreign repository verify-fix Verify if the implemented fix has solved the issue
Projects
None yet
Development

No branches or pull requests

3 participants