Add rework from #67912#1
Merged
jbouwh merged 4 commits intomqtt-notify-correct-setupfrom Mar 12, 2022
Merged
Conversation
jbouwh
added a commit
that referenced
this pull request
Mar 12, 2022
* Move notify serviceupdater to Mixins * Move tag discovery handler to Mixins * fix tests
jbouwh
added a commit
that referenced
this pull request
Mar 12, 2022
* Move notify serviceupdater to Mixins * Move tag discovery handler to Mixins * fix tests
jbouwh
added a commit
that referenced
this pull request
Mar 13, 2022
* Move notify serviceupdater to Mixins * Move tag discovery handler to Mixins * fix tests
jbouwh
added a commit
that referenced
this pull request
Mar 15, 2022
* Move notify serviceupdater to Mixins * Move tag discovery handler to Mixins * fix tests
jbouwh
added a commit
that referenced
this pull request
Apr 1, 2022
* Move notify serviceupdater to Mixins * Move tag discovery handler to Mixins * fix tests
jbouwh
added a commit
that referenced
this pull request
Apr 1, 2022
* Move notify serviceupdater to Mixins * Move tag discovery handler to Mixins * fix tests
jbouwh
added a commit
that referenced
this pull request
Apr 7, 2022
* Move notify serviceupdater to Mixins * Move tag discovery handler to Mixins * fix tests
jbouwh
added a commit
that referenced
this pull request
Apr 15, 2022
* Move notify serviceupdater to Mixins * Move tag discovery handler to Mixins * fix tests
jbouwh
added a commit
that referenced
this pull request
Apr 15, 2022
* Proof of concept * remove notify platform * remove loose test * Add rework from home-assistant#67912 (#1) * Move notify serviceupdater to Mixins * Move tag discovery handler to Mixins * fix tests * Add typing for async_load_platform_helper * Add add entry unload support for notify platform * Simplify discovery updates * Remove not needed extra logic * Cleanup inrelevant or duplicate code * reuse update_device and move to mixins * Remove notify platform * revert changes to notify platform * Rename update class * unify tag entry setup * Use shared code for device_trigger `update_device` * PoC shared dispatcher for device_trigger * Fix bugs * Improve typing - remove async_update * Unload config_entry and tests * Release dispatcher after setup and deduplicate * closures to methods, revert `in` to `=`, updates * Re-add update support for tag platform * Re-add update support for device-trigger platform * Cleanup rediscovery code revert related changes * Undo discovery code shift * Update homeassistant/components/mqtt/mixins.py Co-authored-by: Erik Montnemery <erik@montnemery.com> * Update homeassistant/components/mqtt/device_trigger.py Co-authored-by: Erik Montnemery <erik@montnemery.com> * Update homeassistant/components/mqtt/mixins.py Co-authored-by: Erik Montnemery <erik@montnemery.com> * revert doc string changes * move conditions * typing and check config_entry_id * Update homeassistant/components/mqtt/mixins.py Co-authored-by: Erik Montnemery <erik@montnemery.com> * cleanup not used attribute * Remove entry_unload code and tests * update comment * add second comment Co-authored-by: Erik Montnemery <erik@montnemery.com>
jbouwh
pushed a commit
that referenced
this pull request
Nov 18, 2023
home-assistant#104021) * Fix netatmo authentication loop * Update unit tests * Move logic to determine api scopes * Add unit tests for new method * Use pyatmo scope list (#1) * Exclude scopes not working with cloud * Fix linting error --------- Co-authored-by: Tobias Sauerwein <cgtobi@users.noreply.github.com>
jbouwh
pushed a commit
that referenced
this pull request
Nov 29, 2023
home-assistant#104021) * Fix netatmo authentication loop * Update unit tests * Move logic to determine api scopes * Add unit tests for new method * Use pyatmo scope list (#1) * Exclude scopes not working with cloud * Fix linting error --------- Co-authored-by: Tobias Sauerwein <cgtobi@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Breaking change
Proposed change
Add rework from home-assistant#67912. The discovery update code for the tag platform is shared now,
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: