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 requirements description for SolarEdge Hybrid battery control #16767

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

MarkusGH
Copy link
Contributor

@MarkusGH MarkusGH commented Oct 21, 2024

Fix requirements description.
"Time of Use" sets StorageConf_CtrlMode to 5, which does not work for evcc.

Refs #12129 (reply in thread)

Fix requirements description. 
"Time of Use" sets StorageConf_CtrlMode to 5, which does not work for evcc.
@andig
Copy link
Member

andig commented Oct 21, 2024

Ähhh- Code und Beschreibung widersprechen sich jetzt aber?

@andig andig added the devices Specific device support label Oct 21, 2024
@MarkusGH
Copy link
Contributor Author

Ähhh- Code und Beschreibung widersprechen sich jetzt aber?

Wo denn? StorageConf_CtrlMode wird doch im Code gar nicht verwendet.

@andig
Copy link
Member

andig commented Oct 22, 2024

Arrgh, ich checks erst jetzt. Du ergänzt ja nicht sondern löschst raus. Passt, danke!

@andig andig merged commit 7665094 into evcc-io:master Oct 22, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants