search

scheidtdav / homeassistant-kostal-piko

Home Assistant integration for Kostal Piko inverters
MIT License
14 stars 3 forks source link

Fix Config Flow Data Entry #6

Open scheidtdav opened 1 year ago

scheidtdav commented 1 year ago

During installation, the following dialog is shown: image

It should explain to the user, that host, username and password are supposed to be entered.

As commented here it should also at least explain to include the http schema.

pergolafabio commented 9 months ago

Hi, my old dcs was replaced with a 5.5 BA model, so i needed to use the newer api... Installed this one, but on the config flow the titles were just blanc, so it was a little bit of guessing :-)

Anyway , i think this issue should be reopened? Probability something to do with the dictionaries?

pergolafabio commented 9 months ago

Here is a screen:

Screenshot_20231213-082033

scheidtdav commented 9 months ago

Hey @pergolafabio,

Agreed, that needs to be fixed. Thanks for reporting!

pergolafabio commented 9 months ago

Thnx , and appreciate this version, works quite well!

Only the status, when kostal is "off", I see the status entity in "unknown" state? Shouldn't that be something like power off ?

scheidtdav commented 9 months ago

Only the status, when kostal is "off", I see the status entity in "unknown" state? Shouldn't that be something like power off ?

Well yes and no. I was not able to pin point exactly which state corresponds to what value reported by the inverter. As you can see here there are two unknown values for two different states that the inverter may report. If you happen to be able to say exactly which one is "off" and possibly even what the other one means, I am happy to implement that. In that case, lets move that to a separate issue though :)