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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Assemble platforms upfront in devolo Home Network #80126
Conversation
Hey there @2Fake, mind taking a look at this pull request as it has been labeled with an integration ( |
@@ -106,7 +110,7 @@ async def disconnect(event: Event) -> None: | |||
for coordinator in coordinators.values(): | |||
await coordinator.async_config_entry_first_refresh() | |||
|
|||
await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) | |||
hass.config_entries.async_setup_platforms(entry, platforms(device)) |
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.
This should still be the new async_forward_entry_setups call.
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.
Fixed. This MR was dangling in my personal backlog for quite a while, so it is a good idea to watch out for stuff like this.
c1142cb
to
ae15465
Compare
ae15465
to
45d75ed
Compare
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. |
It's waiting for review ... |
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.
Looks good
Proposed change
In #72030, @balloob suggested to not load all platforms and check for the capabilities of the device in there but to check the capabilities upfront and then only load suitable platforms. This MR addresses that.
Type of change
Additional information
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: