Documentation issue

[lilliesAndRoses]

The documentation of the function updateConnectionParameters says following about the connection parameters object.

{number} min_conn_interval: Minimum Connection Interval in 1.25 ms units.
{number} max_conn_interval: Maximum Connection Interval in 1.25 ms units.
{number} slave_latency: Slave Latency in number of connection events.
{number} conn_sup_timeout: Connection Supervision Timeout in 10 ms units.

If one passes a structure as below the API fails.

var connParams = {
            min_conn_interval: interval,
            max_conn_interval: interval,
            slave_latency: 3, 
            conn_sup_timeout: 4000,
        };

This is because internally a _getConnectionUpdateParams is called which expects

var connParams = {
            minConnectionInterval: interval,
            maxConnectionInterval: interval,
            slaveLatency: 3,
            connectionSupervisionTimeout: 4000,
        }

The connect API on the other hand takes the first structure as input. This is inconsistent and needs fix.

There is one more problem with the connection parameters. The documentation says that it is in the 1.5 ms units, but actually the value is used "AS IS". The documentation needs a fix here as well.

Thanks
Amruta

[bihanssen]

@lilliesAndRoses thanks for reporting. Marking this as a bug.