search

muppet3000 / homeassistant-grott

Home Assistant Integration for Grott - MQTT
MIT License
65 stars 14 forks source link

Added extra note to readme to prevent auto discovery from not working #54

Closed prom00 closed 7 months ago

prom00 commented 8 months ago

I had issues setting this up at first. I used to set my topic in Grott config.ini to 'growatt' instead of 'energy/growatt'. Since this add-on specific only searches in 'energy/growatt', it couldn't find my MQTT data.

I wasn't able to find any note about this, so decided to add it to the readme for now. Might be better if you could configure it in HA.

muppet3000 commented 8 months ago

How would you feel if I just updated the plugin to support a custom string with the default bring what Grott uses by default instead of updating the docs in this way? Would you be willing to test it with me?

prom00 commented 8 months ago

That might be better, and fallback default to energy/growatt, since this is the default in Grott.

Would be even nicer if a user could see a list of available topics, this way the user can see that the MQTT part of the plugin is working.

I've been searching quite alot before I found it was as simple as the topic name.

And ofcourse, if anything needs to be tested, I can do that.