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
32 stars 4 forks source link

Using deep-refresh for polling #43

Closed ajohnston1219 closed 8 months ago

ajohnston1219 commented 8 months ago

https://github.com/derek-miller/homebridge-genie-aladdin-connect/blob/a78269264dda0c0c4afed358beea75dc5a405fa0/src/aladdinConnect.ts#L214

On the above line, you are calling /deep-refresh to get the device status. There is a GET /devices endpoint that would be better for this. You will receive the same device data as the deep-refresh, but it is less expensive on our side. The deep-refresh endpoint actually sends a sync command to the device, which causes it to send relatively large sync response message. It also contains a lot of other information that you are not using.

I know we are working on a new solution, but in the meantime this would really help us out on our end. If you have any questions or need any help just let me know at adam_johnston@geniecompany.com

derek-miller commented 8 months ago

This is fixed in v4.0.1.