[Bug] Docs incorrect about units for "interval"

[chrisjshull]

https://ztalbot2000.github.io/homebridge-cmd4/autoGenerated/CMD4_AccessoryDescriptions.html#interval_D and https://ztalbot2000.github.io/homebridge-cmd4/autoGenerated/CMD4_AccessoryDescriptions.html#interval both claim that "interval" is in seconds. Is it actually in milliseconds?

[chrisjshull]

also, apparently the docs deep linking doesn't work - those are supposed to be the links to the Directive and Accessory Directive pages on "interval"

[ztalbot2000]

Hi, I will have a look. If I remember correctly, interval is in seconds, but it's always good to check again. Ttyl, John

…

On Sun, Jun 4, 2023 at 3:39 PM Chris J. Shull ***@***.***> wrote: also, apparently the docs deep linking doesn't work - those are supposed to be the links to the Directive and Accessory Directive pages on "interval" — Reply to this email directly, view it on GitHub <#139 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABSBCX3KNDSFVQLS3FFMNG3XJTQA5ANCNFSM6AAAAAAY2EHK5A> . You are receiving this because you were assigned.Message ID: ***@***.***>

[ztalbot2000]

Hi,

Sorry, I don't have much time lately. The interval is actually in seconds. If you look at Cmd4Accessory.js under the parseConfig procedure in the current release on line 1581 you will see the line:

case constants.INTERVAL:
// Intervals are in seconds
this.interval = parseInt( value, 10 ) * 1000;

As for the deep link. Under Directives interval does link to:
https://ztalbot2000.github.io/homebridge-cmd4/autoGenerated/CMD4_AccessoryDescriptions.html#interval_D
For whatever reason on Safari, I have to scroll up one line to see the header.
I did add link checks to V7.0.1 and while it did not find anything related to your bug, it did find two for tags that were moved to the Developers documentation. So Thanks for making me double check it all.
I'll be releasing 7.0.1 momentarily. The documentation has already been submitted.

John

[ztalbot2000]

If you have see anything else, please let me know.