-
Notifications
You must be signed in to change notification settings - Fork 502
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
Add support for Danalock v3 #520
Comments
I agree! |
Hey, i tried to integrate my Danalock V3-Zigbee. But the problem is, that the cluster for changing the lock state is shown as "unknown cluster". Attached you can see the information I can see. Also I have attached to official zigbee documentation for the danalock. Additional info: Danalock uses new ZHA V3 and is currently supported only by amazon echo plus (with zigbee) May someone help? Best regards |
You’ll need to add the 0x0101 cluster (attributes and commands) to Next step would be figuring out how the lock responds to the various commands, how its state is reflected in the attributes, and whether it supports attribute reporting. Then, we need to figure out what |
I'm very new to zigbee "coding". May someone is willing to help me :) I guess it would also help others AND! i think when the raspbee supports smartlocks in general it will be a great advantage for the product. |
So I now added the following cluster:
Its adopted from danalocks zigbee documentation (mentioned above). But with this code, every cluster in deconz is now showing as "unknown". I cannot see any syntax errors. May someone has more know how then me? :) Best regards |
If all the clusters show unknown, there’s something wrong with the general.xml. Typically a syntax error, but maybe also a logical error. The command IDs seem to have duplicates. Did you put this in the Closures domain, just before the Windows covering cluster? If you could attach the zipped general.xml, I’ll have a look. EDIT The Danalock Zigbee manual is very brief. Looking at the ZCL spec, the attributes aren’t all uint32 nor read/write. Also, the response commands have a mandatory payload, and the set commands take the PIN as optional payload. Not sure if the Danalock supports a PIN though. |
More information is not submitted by danalock ... Attached you can find the requested zip file containing general.xml Best regards |
Anyone already reviewed my case? Thank you very much 😘 |
Okay. I just had some tries... I compared my general.xml with the "original" one, here from github. The problem still appears. Every cluster is now unknown. Thefuck is wrong? |
Looks like you placed the new entries in the wrong domain.
|
Hi Yes i did. but as i mentioned, i additionaly removed the cluster and compared it with the original. And still everything is unknown. I also let deconz reread the config |
I never got that to work. Best quit and restart deCONZ to reload general.xml. Could you try the general.xml from my commit mentioned above? |
Dang! This works out of the box! Now let's just make looks "apiable". Why we don't disguise it as a switch? Like 1 is locked and 0 is unlocked. |
Could you please post a screenshot of the Door Lock cluster (after reading all the attributes)? Could you check if you can setup attribute reporting for the Lock State attribute? I would hope/expect that this attribute will reflect the current lock state automatically when you issue a Lock Door or Unlock Door command, and when you control the lock manually. |
Indeed, best expose it as a light, with
In light_node.cpp:
That should be enough for the resource to be created (I hope). Best search for Note to self: the device id 0x000a for Door Lock is still missing from general.xml. Next, you need to map To control the lock from the REST API, you need to create a new routine |
How does it work? Where can I do this? |
Follow the steps in the README to get a local copy of the source of the REST API plugin and compile and install it. Then edit your local source and recompile the plugin. |
I am a beginner in these things. Is it possible that you upload the changed files? Thanks in advance!!! |
So guys. I published my plugin changes here: https://github.com/pascal1337/deconz-rest-plugin/ I did many modifications according window_covering and Warnings. I am now at a point, where I'm not able to get further. Compilation still shows a "undeclared"-error and the API won't bring up my lock. Maybe someone can have a closer lo(o/c)k :P |
VENDOR_DANALOCK should be the Manufacturer code from the Node info panel. What is the compile error? |
The following error appears:
|
Yes. This is the point where I currently stand. |
Did you declare the lock and unlock functions in de_web_plugin_private.h? |
Yes I did. Row 1072, 1073 |
You also need to declare the new tasks in deconz-rest-plugin/de_web_plugin_private.h Line 458 in db74fcb
|
Sooo guys. When I make an API request It shows the light correctly with all its details:
When I try to change steht state PUT http://192.168.178.100/api/CD5466EA30/lights/7/state The response is
|
Do we have three options?
My use cases would be:
|
As it's treated as a light bulb, you got on/off (lock/unlock). That Auto Unlock, you already got in the Danalock app, including the "hold on unlock". |
@smulle48 |
So far, it's only possible to lock and unlock through zigbee. The rest you can do from the official Danalock app |
Okay, thank you very much! |
Closing this one as support has been provided with version .79. |
I just bought a Danalock V3, it came with FW v.0.16.0. If your lock state doesn't update automatically
Just like that the status began to update automatically - even when manually turned. |
You could achieve the exact same by just removing the batteries and insert them again. That's exactly what you achieve by updating the firmware, you are rebooting the lock (it's completely cutting the power for a few seconds) after successful update |
@smulle48 oh ok, so you are saying that the bug with the state not being updated in deconz could have just been resolved by restarting the lock? I suspected it was something wrong with the FW or some zigbee params were set incorrectly. |
I suspect it's a design flaw in the hardware itself. I've had both the zwave and now the zigbee version and they both have the same bug. And with both, after inclusion, there was no status before removing the batteries a few seconds from the lock. |
That´s a fantastic idea. Will buy me new batteries tomorrow and try it that way, @InToSSH :-) Don´t think there will be an update from Danalock like they promissed. I saw simular locks in the web with another brand, so i guess they sold it ?! |
I bought batteries allready today and tried to connect my Danalock, but without success :-( |
@Michaelnorge you cannot connect at all? My solution was specific to the state not being updated, otherwise I was connected and able to control the lock. What is your procedure to connect the lock to gateway? I simply paired the lock with Danalock app on my phone via bluetooth. Opened Phoscon app, add click "Add new light", they I went into the settings in Danalock app, chose Smart home, Z-wave (yes it's call Z-wave even if you have Zigbee version), and clicked Connect, then it got connected to my Conbee and appeared as new light in Phoscon app. |
Still no success getting it to work? |
@InToSSH I tried to connect Danalock a lot of time and in a lot of different ways without any success. Even @smulle48 tried to figure out why my Danalock is not connecting to Phoscon. I think It´s time now to give up. Will buy me a shovel and make a nice hole in my garden :-) |
Have you tried the procedure using the GUI ? with reading the basic attributes ? |
Did you also try Zigbee2MQTT instead of Phoscon? :) I´m sorry but my experience showed that I can better work with that instead of phoscon |
@TTP1106 no I haven't tried that, Zigbee2MQTT docs say that support for Conbee is still experimental, so I don't know if everything is supported. EDIT: sorry, just realized you weren't asking me, it just showed up with me being tagged in the message so I replied :) |
So I have no issues to lock/unlock in Home Assistant, that works great. Is it possible to get battery status? |
As it's treated as an light bulb, that's currently not possible |
Unrelated question, if it's treated as a ligh bulb, why don't i get any deconz_event for it in home assistant? |
There is a sensor ZHAbattery just for it nope ? |
I m reading the link. |
Hello. And what about battery status? Can i check battery charge? |
On DDF version it have the state/battery #6057 |
Thanks a lot. Do you use it? Does it stable? |
I don't use it on my side but yes, there is improvement for the device disconnection https://forum.phoscon.de/t/danalock-v3-new-fw-0-20-0-losing-connection/1838 (need firmware update) |
Smart locks are the future, so let's support the Zigbee version of the Danalock v3.
https://danalock.com/danalock-v3.html
The text was updated successfully, but these errors were encountered: