Home Assistant Service Notification Support Added#1294
Open
Home Assistant Service Notification Support Added#1294
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1294 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 197 197
Lines 26067 26166 +99
Branches 4253 4279 +26
=========================================
+ Hits 26067 26166 +99
🚀 New features to boost your workflow:
|
|
Hi, Persistent notifications with the original url still works as expected. Edit: I think the parameter |
5d835e0 to
4ecc90a
Compare
Owner
Author
|
@svenjochems : very sorry for delay; is this still happening using the latest snapshot of this branch? |
4ecc90a to
b6982dc
Compare
Owner
Author
|
Give it another try and let me know how it goes @svenjochems 🙏 |
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
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.
Description:
Related issue (if applicable): #1235
Addition of the following accepted Home Assistant URLs (allowing direct service notifications - to those that support it):
{schema}://{host}/{accesstoken}/{target}{schema}://{host}:{port}/{accesstoken}/{target}{schema}://{user}@{host}/{accesstoken}/{target}{schema}://{user}@{host}:{port}/{accesstoken}/{target}{schema}://{user}:{password}@{host}/{accesstoken}/{target}{schema}://{user}:{password}@{host}:{port}/{accesstoken}/{target}A
targetfollows the following set of logic:You can specify more then one target by just separating it with a
/(slash) on the URL... i.e:hassio://hostname/long.lived.token/my_subsystem/my_subsystem2:targetAChecklist
tox -e lintand optionallytox -e format).tox -e qa).Testing
Anyone can help test as follows: