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 setup entry method for HA 2023.5.0 #79

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Patrick762
Copy link

Possible fix for #76

@mweber89
Copy link

@vwt12eh8, can you please merge?

@MarkWatt3295
Copy link

Any update on this?

@dnelson
Copy link

dnelson commented Sep 11, 2023

Confirmed that this PR fixes the issue for me. :shipit:

@iBobik
Copy link

iBobik commented Oct 14, 2023

Delta Pro was successfully added when applied this PR.

@mrdrmmn
Copy link

mrdrmmn commented Oct 25, 2023

Any ETA on merging this?

@bkhrupa
Copy link

bkhrupa commented Nov 17, 2023

HA: 2023.11.2

I am apply this PR and successfully connected DeltaPro! Hooray!

Please merge!

Error from log without this changes.

Logger: homeassistant.config_entries
Source: config_entries.py:402
First occurred: 1:05:26 AM (1 occurrences)
Last logged: 1:05:26 AM

Error setting up entry DELTA Pro 030141 for ecoflow
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 402, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/ecoflow/__init__.py", line 275, in async_setup_entry
    hass.config_entries.async_setup_platforms(entry, _PLATFORMS)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'ConfigEntries' object has no attribute 'async_setup_platforms'

This was referenced Jan 4, 2024
@adityaselect
Copy link

@vwt12eh8, can you please merge?

Requesting merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants