Fix Devolo cloud connection within API#36158
Conversation
|
@2Fake I've updated the PR description with links to the changelog and release notes. Please make sure you include them next time, as it saves us the need to search for them. Looking at the changes upstream, it is not just a fix, but there is a whole bunch of new features implemented. From question on Discord, I understand you wanted this to go into a patch release. However, I'm not sure if that is a safe thing, considering all the changes. Can you elaborate on that? |
|
@frenck You are right, that the API can do more, but home assistant will not use them, because the integration will just use the binary switches delivered by the API. The other devices will come with another pull request. The important file for this integration in the API is properties/binary_switch_property.py. This file was not touched in this release. The change we need merged into home assistant is in the file backend/mprm_websocket in function _on_pong() |
Breaking change
Proposed change
Bump API version to keep a cloud session alive.
Release notes: https://github.com/2Fake/devolo_home_control_api/releases/tag/v0.11.0
Changelog: 2Fake/devolo_home_control_api@v0.10.0...v0.11.0
Type of change
Example entry for
configuration.yaml:# Example configuration.yamlAdditional information
The issue was fixed in the underlying API see here
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: