Added eTotal for total device production

DiedB / Homey-SolarPanels

Connects production statistics of a wide range of PV inverters to Homey

https://apps.athom.com/app/it.diederik.solar

GNU General Public License v3.0

54 stars 37 forks source link

Added eTotal for total device production #250

Closed 9khil closed 1 year ago

9khil commented 1 year ago

Added this property, but I am not able to test it on a Homey. Please test and pull!

pro-sumer commented 1 year ago

What's your goal?

It looks like you added a property eTotal to the Growatt API, but you’re not using it anywhere?

9khil commented 1 year ago

First of all: sorry I forgot to add the rest of the code.

My goal is to add lifetime energy produced from the API so its available on Homey. I do not have the ability to test it as I do not own a Homey (doing this feature for a friend), so it was a long shot. Sorry if Im wasting your time :D

I just looked at the reguests when visiting growatt and saw that this data was readily available, but Im not sure which API you use.

Please have a look now.

pro-sumer commented 1 year ago

I'm not super familiar with (creating) Homey Apps myself.

If you add a capability like this, will it show up in the App UI? (My guess is you need to change app.json as well and maybe more?)
If you add a capability like this, will it show up in Insights?

Also, if a capability is added, should it be added for all supported brands, or is only a single brand fine?

Maybe @DiedB can chime in on this?