pimoroni / enviro

MIT License
104 stars 83 forks source link

Switch to retained publication for MQTT target #2

Closed andypiper closed 2 years ago

andypiper commented 2 years ago

Propose changing the MQTT publication to retained by default, this makes it easier to see the last reading published from an Enviro sensor if the viewer / consumer / subscriber joins later. There is already a timestamp in the publication message so should be no risk of being confused at whether that reflects the current, or past, status of the Enviro. Added comments to indicate how to switch this back to False if preferred.

As an aside, I'm now comparing the format of these messages to those published by the MQTT sample shipped with the classic Enviro+ HAT, and will probably send a PR for the latter to get the data formats more aligned if possible.

Signed-off-by: Andy Piper andypiper@users.noreply.github.com