notify: add a channel to the home assistant notification #1
Conversation
https://companion.home-assistant.io/docs/notifications/notifications-basic/#notification-channels This passes through to be a specific notification category on Android (I'm not sure how iOS handles these). In the future the notification type (i.e. motion, person, etc) could be appended to the channel name to allow even more specific notification tuning. I didn't see this data in the `NotifierOptions` object passed in.
|
I appreciate the pull request, but I think this really ought to be placed in the Scrypted SDK level, so callers of Notifier specify the channel themselves. Someone else mentioned this feature the other day in Discord, was that you? |
|
Also not sure if this is similar to Android's notification categories and iOS notification channels. The iOS ones need to be defined up front in the iOS app. Is HA doing its own thing here? Ie, this channel is something specific to companion app? |
Ack. In this case would the HA plugin have a static prefix like
Nope :) I just started using Scrypted the day of this PR
Yep, sorry I should have clarified better. I have an automation that publishes with
I see where my confusion came from. Apparently these are Android specific, the HA docs on it have notification channels listed under Android-specific features, so it seems like |
|
Alright looks like I'll need to add this to the SDK, and then callers can provide one, otherwise a default will be used. Will have no effect on iOS. |
This change spins off from conversation at scryptedapp/homeassistant#1 and allows consumers of the `Notifier` interface to specify a channel for notifications to be sent to. Android platforms can use this to send notifications to a specific channel, allowing the user to have fine-grained control over the audio and priority of the notifications they receive.
|
Drafted a PR koush/scrypted#1625 |
This change spins off from conversation at scryptedapp/homeassistant#1 and allows consumers of the `Notifier` interface to specify a channel for notifications to be sent to. Android platforms can use this to send notifications to a specific channel, allowing the user to have fine-grained control over the audio and priority of the notifications they receive.
…1625) * feat(sdk): add support for notification channel in NotifierOptions This change spins off from conversation at scryptedapp/homeassistant#1 and allows consumers of the `Notifier` interface to specify a channel for notifications to be sent to. Android platforms can use this to send notifications to a specific channel, allowing the user to have fine-grained control over the audio and priority of the notifications they receive. * chore(sdk): place Android notification channel under the android object * fix(automation-actions): remove channel from UI
|
published |
…-mapping Add MotionSensor interface mapping for Home Assistant binary sensors
2 participants
https://companion.home-assistant.io/docs/notifications/notifications-basic/#notification-channels
This passes through to be a specific notification category (
scrypted) on Android (I'm not sure how iOS handles these). Prior to this change, the notification comes in on the default Home Assistant Mobile App channelIn the future, the notification type (i.e. motion, person, etc) could be appended to the channel name to allow even more specific notification tuning. I didn't see this data in the
NotifierOptionsobject passed in.Edit: now based on draft koush/scrypted#1625