ericklein / air_quality

displays and logs local indoor and outdoor weather and air quality information
MIT License
0 stars 0 forks source link

Postref influx #44

Closed disquisitioner closed 2 years ago

disquisitioner commented 2 years ago

The code changes here are largely to add support for storing data in an InfluxDB server instance. This introduces the new function post_influx(), which is in its own .cpp file for ease of integration, along with suitable additions to config.h and influx.h (plus the right place in air_quality.ino to call post_influx()).

However, I noticed in looking at the code differences for this pull request there are a lot of flagged differences for whitespace changes. It's possible something happened in a recent merge that altered spacing in numerous lines in air_quality.ino, which weren't present in the starting point I used to create the postref_influx branch used for this feature work.

If you decide to reject this pull request because of all the highlighted whitespace differences I could go back and re-integrated the InfluxDB support in this branch into the current version of air_quality.ino in the main branch. Alternatively you can check all the whitespace differences flagged below and see if you think they are OK such that you can approve this pull request.

Let me know what you prefer.