search

thomascys / solaxcloud

SolaxCloud integration for Home Assistant.
22 stars 11 forks source link

Use getInverterInfo #3

Closed pieterzen closed 4 years ago

pieterzen commented 4 years ago

Hi,
Great that you've shared your solution to the Solax API debacle. I've been trying to a Rest call in HA but the archaic sessionId stuff is not supported in HA. I will try your plugin, but I already have a suggestion. Instead of DailyInfo (which is huge) you can use the following endpoint: http://www.solaxcloud.com:6080/proxy/mysite/getInverterInfo?siteId=<SiteId>&tokenId=<tokenId>

This call only returns the current data for your inverter. You need a Site, and then the siteid which you can find inspecting the link to your site.

thomascys commented 4 years ago

Just tested that endpoint. This is amazing and would mean an enormous performance boost for this integration. I'll try to rewrite it one of the coming days. Thanks again for this great suggestion! I wasn't aware of that endpoint.

pieterzen commented 4 years ago

I've already done that and tested it. If you can give me permissions I can push a branch so you can have a look too.

thomascys commented 4 years ago

Feel free to provide a pull request and then we can move on from there

pieterzen commented 4 years ago

Took me a while to work that out :) Don't work with external repo's that much

thomascys commented 4 years ago

Reviewed your pull request and merged it. I've also added new installation instructions.

Thanks for taking the time and share this improvement!