toto / homebridge-airrohr

HomeBridge module for the DIY luftdaten.info particulates sensor. See http://luftdaten.info how to build your own
http://luftdaten.info
MIT License
52 stars 12 forks source link

Make public temperature optional #2 #22

Closed nicoh88 closed 6 years ago

nicoh88 commented 6 years ago

Thanks @alexryd

Copy & Paste from PR19 with index.js conflict fix.


This PR makes the public_temperature_json_data and public_airquality_json_data config variables independent of each other so that it's possible to only load air quality data without temperature data or the other way around. So for instance if public_temperature_json_data is not specified in the config, no TemperatureSensor or HumiditySensor services will be exposed and no temperature data will be loaded. Fixes #16 and fixes #15.

nicoh88 commented 6 years ago

@alexryd Thanks! @toto Please merge, push to npm and close #15, #16, #19 and #22.

toto commented 6 years ago

Very nice Thx @nicoh88!

nicoh88 commented 5 years ago

@toto Can you publish npm 1.1.7? Thanks

toto commented 5 years ago

Done

nicoh88 commented 5 years ago

Hmm 🤔.

No „npm publish“?

https://www.npmjs.com/package/homebridge-airrohr

toto commented 5 years ago

Caching i guess. Give it a moment. Bumped to 1.1.8 fixing security problems in dependencies.