-
Notifications
You must be signed in to change notification settings - Fork 44
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
Integration broken in Home Assistant 2023.5.0 #76
Comments
Tested it on my Home Assistant with Ecoflow River Max |
Confirmed the same issue here on 2023.5.0 @vwt12eh8 can you merge @Patrick762 fix? |
Same issue on my Home Assistant Yellow with the DELTA Pro. |
@Patrick762 your fix seems to work with the latest alpha version. |
@HHoppenstock Can you try again? There was a method missing for the EcoFlowExtraDevice I'll try to prepare a PR for the latest stable version now |
@Patrick762 Now all entities are gone: |
Can you tell me if there's any error in your logs for the integration? |
@Patrick762 Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht Logger: homeassistant Error doing job: Task exception was never retrieved |
That's my updated init.py which causes the error above |
I think the main branch should be updated first then. |
@Patrick762 @lwsrbrts thanks for your great work and assistance. It's running fine now. |
This appears to have fixed the issue for me also. Thanks. |
Have two River Max connected without issues for months. |
Thanks, this also worked for me on the stable version. River Pro |
This fix works for my Delta Pro. Would love to see the PR accepted into the stable branch since it breaks "normal" users experience. |
Also: FYI - The integration is still working with DP firmware 1.0.1.49 (WiFi = 0.1.0) |
Change line 24: "hass.config_entries.async_setup_platforms(entry, _PLATFORMS)" by "await hass.config_entries.async_forward_entry_setups(entry, _PLATFORMS)" References: vwt12eh8/hassio-ecoflow#76 https://community.home-assistant.io/t/issues-with-nest-protect-after-installing-20230503-1/567537/6
I have just updated to core 2023.5.4 from 2023.3.6 same issue - fix above resolved the issue. /config/custom_components/ecoflow/init.py line 275 `Logger: homeassistant.config_entries Traceback (most recent call last): EcoFlow Delta Pro running firmware V1.0.1.49 / V0.1.0 (Wi-Fi) |
I ahve made the changes to the file mentioned above it gets rid of errors in log but there are no devices or entites. I have a delta pro and support say they have fixed api issue after firmware update. Any ideas? |
I believe it is the latest WiFi update that closed the local TCP port. This integration does not use sanctioned access or a public API supported by Ecoflow. It is basically a private API written as a custom HACS integration for HA. If port 8055 is not open on your DP you will need to ask support to reopen that port in the next WiFi update (and roll back your WiFi firmware). |
they assure me its on the latest version which has it open. |
Who is "they" and can you confirm the port is open (with nmap, or telnet, or a script that opens the port and receives data on it)? If you can verify it is closed you can go back to them and ask them to re-open it or revert your firmware. It may have gotten closed inadvertently or perhaps it was intentional. Either way I suspect the main support people don't know unless they talk to the person or team that actually made the change. Last time this happened it had nothing to do with the main system firmware (.61) it was the wifi update that closed the port (for certain units with a particular wifi chip). |
i decieded to start with a completely fresh install of HA and then started again. Once i changed the error line of code above it started working. |
It looks like WiFi firmware 3.0.2.25 has closed TCP port 8055. Anyone have any info on other versions or if EF will re-open the port again in subsequent firmware? |
With this change, I now get:
|
I got the following error in the latest Home Assistant Release:
It seems like it can be fixed by changing the line from
to
The text was updated successfully, but these errors were encountered: