homebridge-eufy-security / plugin

Homebridge plugin to control certain Eufy Security devices
Apache License 2.0
158 stars 24 forks source link

[enhancement]: Enable Smart Lock C210 (T8502; E110) compatibility in plugin #523

Open codychamberlain opened 3 months ago

codychamberlain commented 3 months ago

Describe the solution you'd like

According to [https://bropat.github.io/eufy-security-client/#/supported_devices] it looks as though Smart Lock C210 (T8502; E110) is fully supported.

The plugin, however, doesn't seem to have the lock enabled:

[03/11/2024, 11:46:27] [EufySecurity] INFO Registering new accessory: Patio Door [03/11/2024, 11:46:27] [EufySecurity] INFO Registering new accessory: Front Door [03/11/2024, 11:46:29] [EufySecurity] WARN This accessory is not compatible with HomeBridge Eufy Security plugin: Patio Door Type: 180 [03/11/2024, 11:46:29] [EufySecurity] WARN This accessory is not compatible with HomeBridge Eufy Security plugin: Front Door Type: 180 [03/11/2024, 11:46:29] [EufySecurity] ERROR Patio Door handleSecuritySystemCurrentStateGets: Wrong return value [03/11/2024, 11:46:29] [EufySecurity] ERROR Patio Door handleSecuritySystemTargetStateGet: Wrong return value [03/11/2024, 11:46:29] [EufySecurity] ERROR Front Door handleSecuritySystemCurrentStateGets: Wrong return value [03/11/2024, 11:46:29] [EufySecurity] ERROR Front Door handleSecuritySystemTargetStateGet: Wrong return value

github-actions[bot] commented 3 months ago

Did you check this Road Map pages ?

codychamberlain commented 3 months ago

Road Map doesn't seem to be working. Apologies if this is hosted somewhere else and I missed it.

slowpc1 commented 3 months ago

Looks like the locks were just added this month 3/3 to the device list there at the start of the month. Hopefully it will be updated and added to function correctly in Homekit. (Just got one myself and was wondering about support)

Antergosgeek commented 3 months ago

I just recently bought a C210 (E110) myself. It currently shows up and I can actually lock/unlock in the HomeKit app. Only thing that isn’t working: it doesn’t show the status of the lock correctly. This is on the latest beta of the plugin 2.3.11. .

lenoxys commented 3 months ago

is the status inversed ? can you described more the expected status ?

slowpc1 commented 3 months ago

is the status inversed ? can you described more the expected status ?

Current status is the devices are not supported in the latest build plugin , just the database of devices some of the plugin is based on is not updated to reflect it.

I believe.

lenoxys commented 3 months ago

did you tried on 2.3.11+

slowpc1 commented 3 months ago

I ran a few betas a few days ago up to latest at that time. Not in front of container so can’t give exact release number but will update today when available.

Antergosgeek commented 3 months ago

is the status inversed ? can you described more the expected status ?

The lock is not inverted it shows correctly if open or closed when I click on at it but says „UNKNOWN“ and shows an open symbol in the HomeKit Dashboard even though when I click on the tile it then shows that it is locked which is also the actual state of the lock. See pictures. IMG_8599 IMG_8598

Hope this helps. FYI my lock is called „Front Door“.

Antergosgeek commented 3 months ago

IMG_8600

Antergosgeek commented 3 months ago

Sorry for the multiple Comments, best I can do on phone. IMG_8601

@lenoxys: I am also getting a Video Doorbell S220 soon, if that’s of interest for any sort of feedback.

lenoxys commented 3 months ago

unlock status should be fixed on 2.3.14. video streaming is still broken.

codychamberlain commented 3 months ago

Running version 2.3.14 - the locks are identified and am no longer getting a "not compatible error" but still getting this error: [03/19/2024, 16:48:49] [EufySecurity] INFO Updating existing accessory: Patio Door [03/19/2024, 16:48:49] [EufySecurity] INFO Updating existing accessory: Front Door [03/19/2024, 16:48:52] [EufySecurity] ERROR Patio Door handleSecuritySystemCurrentStateGets: Wrong return value [03/19/2024, 16:48:52] [EufySecurity] ERROR Patio Door handleSecuritySystemTargetStateGet: Wrong return value [03/19/2024, 16:48:52] [EufySecurity] ERROR Front Door handleSecuritySystemCurrentStateGets: Wrong return value [03/19/2024, 16:48:52] [EufySecurity] ERROR Front Door handleSecuritySystemTargetStateGet: Wrong return value

Antergosgeek commented 3 months ago

unlock status should be fixed on 2.3.14. video streaming is still broken.

Mine is working perfectly now shows status correctly and everything.

IMG_8616 IMG_8617

lenoxys commented 3 months ago

Can you open a new issue for this thanks ?

And provide the full logs ?

Running version 2.3.14 - the locks are identified and am no longer getting a "not compatible error" but still getting this error:

[03/19/2024, 16:48:49] [EufySecurity] INFO Updating existing accessory: Patio Door

[03/19/2024, 16:48:49] [EufySecurity] INFO Updating existing accessory: Front Door

[03/19/2024, 16:48:52] [EufySecurity] ERROR Patio Door handleSecuritySystemCurrentStateGets: Wrong return value

[03/19/2024, 16:48:52] [EufySecurity] ERROR Patio Door handleSecuritySystemTargetStateGet: Wrong return value

[03/19/2024, 16:48:52] [EufySecurity] ERROR Front Door handleSecuritySystemCurrentStateGets: Wrong return value

[03/19/2024, 16:48:52] [EufySecurity] ERROR Front Door handleSecuritySystemTargetStateGet: Wrong return value