derek-miller / homebridge-genie-aladdin-connect

Homebridge plugin to integrate Genie Aladdin Connect enabled garage door openers.
https://www.npmjs.com/package/homebridge-genie-aladdin-connect
Apache License 2.0
30 stars 4 forks source link

Battery Level Illegal Value #17

Closed bwatz001 closed 2 years ago

bwatz001 commented 2 years ago

Describe The Bug: The battery level characteristic is throwing an illegal argument of -1.

To Reproduce: Update plugin to v 2.3, and restart the plugin.

Expected behavior: Should show a valid argument for the battery level to expose in Homebridge / Homekit.

Logs:

6/29/2022, 11:12:04 AMHOOBS BridgeERROR[HOOBS Bridge B57C@My Door@My Door@Battery Level] characteristic was supplied illegal value: number -1 exceeded minimum of 0

Plugin Config:

{ "accessories": [], "platforms": [ { "platform": "GenieAladdinConnect", "name": "Garage Door", "userInfoCacheTtl": 3600, "doorStatusStationaryCacheTtl": 15, "doorStatusTransitioningCacheTtl": 15, "doorStatusPollInterval": 15000, "username": , "password": } ] }

Screenshots:

Environment:

derek-miller commented 2 years ago

I do not have a battery sensor to test with. Could you run homebridge in debug mode and set the plugin config value logApiResponses to true then paste in the logs? Make sure to redact any sensitive info. I just need to figure out what the plugin is doing and what the responses look like that have battery info.

derek-miller commented 2 years ago

@bwatz001 I think I fixed the issue, but I would still like to to see the response bodies for a system with battery devices if you have time. Thanks!