Closed mib1185 closed 1 week ago
Hey there @hacf-fr, @quentame, mind taking a look at this pull request as it has been labeled with an integration (synology_dsm
) you are listed as a code owner for? Thanks!
Looks right to me. Tagged since we are having timeout issues.
Will check on my production system shortly and approve if all is well
thanks for additional tests with using cameras 👍
Proposed change
we already setup the api in
https://github.com/home-assistant/core/blob/f5394dc3a3d023b3cb22dafc8d8435679fcda408/homeassistant/components/synology_dsm/__init__.py#L66-L69
so the first data fetch of the central coordinator is not needed and just leads to a duplicated fetch of all data at startup
prior this change:
with this change
cc @bdraco
Type of change
Additional information
Checklist
ruff format 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
.To help with the load of incoming pull requests: