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

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

Open
codychamberlain opened this issue Mar 11, 2024 · 15 comments
Labels
enhancement New feature or request

Comments

@codychamberlain
Copy link

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

@codychamberlain codychamberlain added the enhancement New feature or request label Mar 11, 2024
Copy link

Did you check this Road Map pages ?

@codychamberlain
Copy link
Author

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

@slowpc1
Copy link

slowpc1 commented Mar 14, 2024

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
Copy link

Antergosgeek commented Mar 17, 2024

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
Copy link
Contributor

lenoxys commented Mar 17, 2024

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

@slowpc1
Copy link

slowpc1 commented Mar 17, 2024

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
Copy link
Contributor

lenoxys commented Mar 17, 2024

did you tried on 2.3.11+

@slowpc1
Copy link

slowpc1 commented Mar 17, 2024

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
Copy link

Antergosgeek commented Mar 17, 2024

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
Copy link

IMG_8600

@Antergosgeek
Copy link

Antergosgeek commented Mar 17, 2024

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
Copy link
Contributor

lenoxys commented Mar 19, 2024

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

@codychamberlain
Copy link
Author

codychamberlain commented Mar 19, 2024

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
Copy link

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
Copy link
Contributor

lenoxys commented Mar 20, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

4 participants