-
-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Suggestion: HA add-on #99
Comments
I'm going to play around with this at https://github.com/austinmroczek/addon-solarman-mqtt, following https://github.com/home-assistant/addons-example |
I have a working prototype at https://github.com/austinmroczek/addon-solarman-mqtt if anyone wants to give it a try. |
Nice work, @austinmroczek! I was experimenting with this as well. My thought was, when this becomes an add-on, then the MQTT part becomes obsolete, as it is possible to create/update native HA sensors directly. So, where MQTT now is used to get the Solarman data in HA, an add-on should provide a more native way of doing the same. |
We could make it into an HA Integration, but then we would need to know how to handle various kinds of Solarman logger equipment. Or we would have to make integrations for each type of equipment. For example, I'm doing this to read data from my NeoVolta battery. I think for at least the near future, we'll want to keep MQTT and have users create template sensors like you currently have set up. |
I think there is a lot we could do to simplify setup for users. All of those |
Has anyone made this into a Hass OS add-on ? https://developers.home-assistant.io/docs/add-ons/. Seems like you're 90% of the way there since you have a working container.
I'm willing to figure it out and submit a PR. Just wanted to ask first in case someone had already done it. I'm not seeing it, but may be using the wrong search terms.
The text was updated successfully, but these errors were encountered: