-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Yale YRD210 wrong battery reading #1102
Comments
Koenkk
added a commit
that referenced
this issue
Mar 28, 2020
Koenkk
added a commit
that referenced
this issue
Mar 28, 2020
Koenkk
added a commit
that referenced
this issue
Mar 28, 2020
Yep! I agree the devices are not meeting Zigbee spec. When I said implementation I meant the specific definition in |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
My Yale YRD210 locks (I have four) are correctly detected and the functionality is fine. But the battery reading is incorrect, it reads half what it should. I tested with new batteries and it reports exactly 50%.
zigbee-herdsman-converters/devices.js
Line 6285 in fa037d5
Changing that line to the same as YRD220 like so:
zigbee-herdsman-converters/devices.js
Line 6317 in fa037d5
works and fixes the battery problem.
I'm not sure who wrote that original code, and GitHub blame isn't working (I get a Unicorn). Is it safe to assume that my four locks are correct and the original implementation is not? I'm not sure...
The text was updated successfully, but these errors were encountered: