Hey, thanks for your great work - I have started using this branch and I very much like it.
However I would like propose update of config.ini.dist:
In config.ini.dist there is the following information:
# The MQTT base topic to publish all Mi Flora sensor data topics under.
# Default depends on the configured reporting_method
#base_topic = miflora # Default for: mqtt-json, mqtt-smarthome
#base_topic = homie # Default for: mqtt-homie
#base_topic = homeassistant # Default for: homeassistant-mqtt
#base_topic = v1/devices/me/telemetry # Default for: thingsboard-json
#base_topic = # Default for: wirenboard-mqtt
So as log as I use mqtt-json the default toppic should be miflora.
While the default toppic is no longer **miflora** but **misensor** - what makes total sense cause we no longer use only miflora sensors - also it was already implemented in miflora-mqtt-daemon.py:
if reporting_mode == 'mqtt-homie':
default_base_topic = 'homie'
elif reporting_mode == 'homeassistant-mqtt':
default_base_topic = 'homeassistant'
elif reporting_mode == 'thingsboard-json':
default_base_topic = 'v1/devices/me/telemetry'
elif reporting_mode == 'wirenboard-mqtt':
default_base_topic = ''
else:
default_base_topic = 'misensor'
Hope you dont mind my changes
* switched library 'Mijia Bluetooth Temperature Smart Humidity' from mitemp_bt to mithermometer
* fixed schema generation for openHAB
* set 'retain=True' for mqtt messages for homeassistant-mqtt
* other minor fixes
* posibility to upload MQTT data to thingsboard
* use reconnect for thingsboard-json
* update readme and config example
* Update README.md
* Update config.ini.dist
* Update config.ini.dist
* give hint to install JSONPath Transformation
great guide, but i had not installed JSONPath Transformation (on my basic install), so the generated items did not work...
* Update README.md
* Update README.md
* Update README.md
* Added config dir to simplify docker handling
* Added a Dockerfile
* Implement suggested fix for issue #39
* Improved documentation
* Added documentation on homeasssistant-mqtt reporting mode
* Use local directory as default for config
* Update README.md
* Docs for `homeassistant-mqtt` reporting method
The `homeassistant-mqtt` is already implemented and is a valid `reporting_method` value, but it was not documented.
Adding a brief explanation of it in the example `config.ini`. I realized that the project `README.md` already mentions Home Assistant.
* Update config.ini.dist
Current implementation was failing if the config.ini included an explicit `false` --it was read as a string and thus a `'false'` as a string was `get`ted and it was considered `True`.
The `enabled` flag of the daemon was correctly using `getboolean` already.
Toni Corvera
Oleksandr Kurbatov
Christoph Wempe
wojciej
aqualx
Markkuuss
Artur Neumann
krikk
Skatun
LarsAC
William Hughes
Alex Barcelo
muhkuhgh
chrostek
Norbert Bartels
Thomas Dietrich
William Hughes
Sebastian Raff
nohum