Skip to content
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

Fix Config Flow Data Entry #6

Open
scheidtdav opened this issue May 21, 2023 · 5 comments
Open

Fix Config Flow Data Entry #6

scheidtdav opened this issue May 21, 2023 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@scheidtdav
Copy link
Owner

During installation, the following dialog is shown:
image

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

@scheidtdav scheidtdav self-assigned this May 21, 2023
@scheidtdav scheidtdav added the bug Something isn't working label May 21, 2023
@pergolafabio
Copy link
Contributor

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
Copy link
Contributor

Here is a screen:

Screenshot_20231213-082033

@scheidtdav
Copy link
Owner Author

Hey @pergolafabio,

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

@scheidtdav scheidtdav reopened this Dec 13, 2023
@pergolafabio
Copy link
Contributor

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
Copy link
Owner Author

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 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants