-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
Add warning to SolarEdge local documentation #13125
Conversation
…is component will not work on many inverters drobtravels/solaredge-local#24
Co-Authored-By: Klaas Schoute <klaas_schoute@hotmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be an alert on alerts.home-assistant.io?
Thanks for reviewing. I'm unclear if any action is required by me on this? I see changes have been requested, but it looks like they were already applied. I can add an alert to the PR if you would like. I was unfamiliar with that site. Note that this has been an issue for a long time, but I just got around to adding an update. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @drobtravels!
I've adjusted a bit and moved the warning a bit down (let's start with an introduction before serving the user a warning 😉 )
This integration no longer functions with recent firmware updates. It may continue work long term on inverters which do not receive firmware updates
More details in this issue
Proposed change
Add warning and reference to issue with more details so users can quickly determine if this integration will work for them.
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.