Add Apprise notification integration#26868
Conversation
|
Please don't squash commits after review has started to make it easier for readers to track changes. |
MartinHjelmare
left a comment
There was a problem hiding this comment.
Looks good!
Can be merged when build passes.
|
@MartinHjelmare Thanks for your help and patients in getting the testing working with me. I've updated the docs here in this PR: home-assistant/home-assistant.io#10451 to reference the Quick question though: In Apprise it's referenced as tagging. Do you prefer use of the Thoughts? |
|
We have to use the target argument since that is what the service call may provide. |
|
Great! |
Description:
Added notification support for Apprise which drastically increases the number off notification services supported by Home Assistant. It also, unfortunately, provides some over-lap to services already supported.
Docs PR:
home-assistant/home-assistant.io#10451
Example entry for
configuration.yaml(if applicable):Configuration files can also be sourced (via another location on the same server, or a remote server. Apprise supports TEXT and YAML based configuration formats.
Apprise URLs loaded directly (using the
urldirective) or through configuration files are all appended together in one list. You can use bothconfigandurlin the same configuration if you wish.Checklist:
tox. Your PR cannot be merged unless tests passIf user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
python3 -m script.hassfest.requirements_all.txtby runningpython3 -m script.gen_requirements_all..coveragerc.If the code does not interact with devices: