sensortag - support for the TI Sensortag 2 about node-red-nodes HOT 8 CLOSED

There's a pull request here (please don't merge it! but it seemed to be the best way to share the change, it's not ready yet).

#108

It adds support for the luxometer (light meter) reporting on sensorTag/lux.

from node-red-nodes.

Hi, thanks for this.

AS per our contribution guidelines, we need a signed CLA before we can accept pull-requests - https://github.com/node-red/node-red/blob/master/CONTRIBUTING.md#contributor-license-agreement

As a general comment, you can just post a link to your repo/branch/commit if you want us to look at something - the pull request isn't needed until the code is read to merge.

from node-red-nodes.

Thanks, I'll get the CLA done. I wasn't quite sure of the best way to proceed to notify you all, I'll hold off on further pull requests until everything else is ready.

from node-red-nodes.

Closing this as have recently updated the sensorTag node to support the newer tag also...
(but still not multiple tags at once)

from node-red-nodes.

Thanks. I think that multiple tags (or multiple BLE devices in general) is a harder issue

from node-red-nodes.

yes - I think there is some progress in the library but it's not a done deal... so as this issue was about tag2 and that now works we'll leave this closed and can open a new issue for multiple tags... if required.

from node-red-nodes.

There is an existing issue #109 that was closed, @dceejay - take a look at that and see if it's reopenable, or I can write up a new one that reflects the broader issue of managing multiple connections to multiple diverse BLE devices at the same time.

from node-red-nodes.

As noted on #109, IBM has this problem addressed in an IBM-IoT contribution at. https://github.com/ibm-iot/node-red-contrib-sensortag . (just to bring this issue to a satisfying close)

from node-red-nodes.