Improve IQVIA data/API management based on enabled entities#32291
Improve IQVIA data/API management based on enabled entities#32291bachya merged 7 commits intohome-assistant:devfrom
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #32291 +/- ##
======================================
Coverage ? 94.74%
======================================
Files ? 772
Lines ? 55935
Branches ? 0
======================================
Hits ? 52995
Misses ? 2940
Partials ? 0Continue to review full report at Codecov.
|
There was a problem hiding this comment.
async_added_to_hass should not trigger an update.
There was a problem hiding this comment.
Without this, the entities show unavailable until their next update cycle. With this "API registration" mechanism in play, I'm unclear on how to instruct an entity to update immediately after its data is pulled (which occurs when "interest" is registered first) without this.
adcafd3 to
0025ce4
Compare
ac1b948 to
c41edf1
Compare
balloob
left a comment
There was a problem hiding this comment.
Ok to merge when last comment addressed 👍
Proposed change
#32223 deprecated monitored conditions within IQVIA. Since the integration derives its data from several API calls, this PR goes a step further and improves its data management so that as entities are enabled/disabled, the appropriate API calls are also enabled/disabled.
Additionally, this PR removes some useless warning messages.
Type of change
Example entry for
configuration.yaml: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: