Skip to content

Instantly share code, notes, and snippets.

@sstratoti
Last active April 15, 2024 00:49
Show Gist options
  • Save sstratoti/8021c5a4ee8e34313c3f59ba20c4a83a to your computer and use it in GitHub Desktop.
Save sstratoti/8021c5a4ee8e34313c3f59ba20c4a83a to your computer and use it in GitHub Desktop.
iOS Subflow for HomeAssistant Companion notifications
@sstratoti
Copy link
Author

OK I've updated the flow. Even though URI wasn't populated, it was sending in an empty string, which was causing the default behavior of "foreground" when it is populated.
I also fixed an issue where if msg.notificationOverride wasn't populated, it would send a message twice! Weird bug with my settings on the switch node. Give this update a try? I've edited the gist with the latest.

@sstratoti
Copy link
Author

sstratoti commented Aug 19, 2021

Oh wait - needed to update once more to remove a debug. Should be good....now!

@Post404
Copy link

Post404 commented Aug 19, 2021

I've tested it and it works perfectly!

@Post404
Copy link

Post404 commented Oct 3, 2021

I am using this subflow for a lot of things now and it is working perfect, but while fixing some errors from the log I noticed that this subflow is generating errors (i just cleared it so there is only 1 occurrence):

Logger: homeassistant.components.websocket_api.http.connection
Source: components/websocket_api/connection.py:89
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 22:40:40 (1 occurrences)
Last logged: 22:40:40

[547425187728] Received invalid command: mobile_app/push_notification_channel

I don't think this is a problem, but i'm thinking it might cause one when updates happen. Do you have this error aswell?

@sstratoti
Copy link
Author

Subscribe to this issue:
home-assistant/iOS#1700

From the looks of it, I don't believe it hurts anything.

@Post404
Copy link

Post404 commented Oct 4, 2021

Thanks, that explains it, i've subscribed to it.

@fireone23
Copy link

fireone23 commented Oct 18, 2021

Feel silly asking this question, but I have read through this thread a few times now, I keep getting a 'no connection' error message on the node. Was expecting at least getting a notification as I was getting using the android version of the same subflow. I figure I'm doing something wrong with the config, I have tried both using both notify and mobile_app_eddie as the notify service, neither worked. Can anyone please assist?


Screen Shot 2021-10-18 at 3 50 43 PM

Screen Shot 2021-10-18 at 3 51 08 PM

@pistak23
Copy link

Hello, thank you for this node, but I have a problem with Custom Sound (Optional - Will Override Pre-Installed Selection). When I write own sound, from system sound (I imported them, already) still play only Pre-Installed Custom Sound... Have you the same problem? Or I have some mistake? thank you very much

Snímka obrazovky 2021-10-26 o 8 22 15
.

@sstratoti
Copy link
Author

Initial thought - never heard of a .caf file before. I think the delivered sounds are all wavs?

You might want to check the iOS app documentation to see what is supported?

If supported let me know and I’ll take a deeper look?

@pistak23
Copy link

pistak23 commented Oct 26, 2021

I used this sound before, and works well... Now with this node they dont work. look at this, please. It is screen from Home assistant website.
This is a part of code, and play .caf sound now. Only this node don't play custom sound:
"sound": {
"name": "3rd_Party_Critical_Haptic.caf",
"critical": 0,
"volume": 1
}

And the .waw format I also have, and no works too :(

Snímka obrazovky 2021-10-26 o 13 12 30
Snímka obrazovky 2021-10-26 o 13 11 46

@pistak23
Copy link

Hello again,
I tried to repair this mistake, and I think I done it. In subflow in node function "create service call" I replaced "customSoundPreInstalled" and "customSound" and now it works. Please check it and if is correct, update this node for other people. Thank You

Snímka obrazovky 2021-10-27 o 0 25 53

@fireone23
Copy link

Feel silly asking this question, but I have read through this thread a few times now, I keep getting a 'no connection' error message on the node. Was expecting at least getting a notification as I was getting using the android version of the same subflow. I figure I'm doing something wrong with the config, I have tried both using both notify and mobile_app_eddie as the notify service, neither worked. Can anyone please assist?

Screen Shot 2021-10-18 at 3 50 43 PM

Screen Shot 2021-10-18 at 3 51 08 PM

I figured out the issue. For the call-service nodes in the subflow, I have multiple instances of the Home Assistant server listed, changing that around and found one that allowed the connection.

Screen Shot 2021-10-26 at 3 38 51 PM

@sstratoti
Copy link
Author

@pistak23 - thanks for catching that. The default value is set to "default" which is obviously > 0, so the custom sound would never hit in the logic. Swapping those variables will have it check the custom first, then the default sounds - so that looks good to me. I'm making a correction and will edit the gist to update it.

@sstratoti
Copy link
Author

@fireone23 glad to hear its working now! I've run into that issue myself before, it's frustrating til you figure it out. :)

I believe that you can edit the servers that are available, and remove the dead ones from the list. I think thats what I did long ago...

@henriklund
Copy link

Since 2021.10 iOS actionable notifications allow icons to be included. Mac owners can find the list of SF symbols here. For Windows user, one could look at this list (the list will only show the names, and may not be complete).
Please note that while this works fine on the iPhone, the current versions of WatchOS (even v8.3) will not show such icons.

Following changes should be made to the code:

'create service call' (--> indicates line added)

        const destructive = env.get(`${name}Destructive`);
        const behavior = env.get(`${name}Behavior`);
-->     const icon = env.get(`${name}Icon`);
        
        const actionObject = {};
        actionObject.action = action;
        if (destructive != "") actionObject.destructive = destructive;
        if (behavior != "") actionObject.behavior = behavior;
-->     if (icon != "") actionObject.icon = icon;

        actions.push(actionObject);
    }

Following should be added (in the actionable-notifications-subflow-for-ios.json) after each of the action_TextInputPlaceHolder (where _ is the number 1-4):

{ "name": "action1Icon", "type": "str", "value": "", "ui": { "label": { "en-US": "Action 1 Icon (Optional)" }, "type": "input", "opts": { "types": [ "str", "env" ] } } },

Just remember to change "action1Icon" and "Action 1 Icon (Optional)" accordingly ;)

@fslef
Copy link

fslef commented Nov 17, 2021

Is it possible to set notifications properties from the msg.payload ? (such as the message title or message text)

@DeltaNu1142
Copy link

This subflow is great. I had originally posted a question about a problem I was having, but worked it out. Thanks for putting this together.

@sstratoti
Copy link
Author

sstratoti commented Nov 18, 2021

@fslef - Yes, this is possible. What I do is place a function node in between whatever node is pushing the payload (say an events:state node, or trigger:state).

For instance, here is a flow that I use for my litter robot.

image

...the status code switch is checking the various codes to see if they match any of the ones that I care about.
The "Set Message" function node is what we're interested in.

// set msg overrides
msg.notificationOverride = {};
msg.notificationOverride.title = "Litter Robot error message!";
msg.notificationOverride.message = "Litter Robot is reporting '" + msg.data.attributes.status + "' (" + msg.data.attributes.status_code + "). Please check the litter robot.";
return msg;

You can override the default properties that you've set for the service call by passing in a msg object with notificationOverride set.

// set msg overrides
msg.notificationOverride = {};
msg.notificationOverride.title = "dynamic title!";
msg.notificationOverride.subtitle = "dynamic subtitle";
msg.notificationOverride.message = "Some new message like Battery is at " + msg.payload.attributes.battery + "%";
msg.notificationOverride.services = "my_phone, partners_phone, kids_phone"; //could override this depending on who is "home".
msg.notificationOverride.tag = "xyzTag" // could be that you want to send the same message, but have it appear multiple times on the device rather than overwritten.
msg.notificationOverride.clear = true; // will attempt to clear all messages to all specified devices in the setup or in msg.notificationOverride.services if overridden. I believe this currently isn't working because apple isn't allowing them to do this with the app. But they've petitioned for it... 
return msg;

All of these are optional. If you send in a value, it'll override the default values set in the node so that its more dynamic.

Also, I use the "msg.notificationOverride.services" to dynamically set who gets a message based on who is home, the time of day, if we're asleep, etc. I do this all using another subflow called "Who to send messages to?". It helps to keep things really clean and doesn't bother my wife while she's at work (for instance).

Glad you figured it out @DeltaNu1142 !

@sstratoti
Copy link
Author

Also, thank you @henriklund - gist is updated!

@fslef
Copy link

fslef commented Nov 18, 2021

Thanks @sstratoti

Ok I see. So everything should go through the msg.notificationOverride

Would you mind if you share your flow ?
So that I can have a look on the other nodes ?

Thanks a lot

@bonanitech
Copy link

@sstratoti, do you plan to publish this as a subflow module?

@sstratoti
Copy link
Author

Hi @fslef - I’ll see what I can do next week.

@bonanitech - had no idea about this! Very cool, and this would be much easier for people to receive updates to the node! I’ll also look into this next week as well.

@bonanitech
Copy link

Awesome! It may also be a good idea to scrub the flow with this tool before that. 😉

@sstratoti
Copy link
Author

sstratoti commented Nov 21, 2021

@bonanitech Just did a diff after using that tool, had no idea that the server ID was in there, but it makes sense that it would be. The HA nodes need to use something to connect...

Is there a way to set that dynamically with the subflow module? Or all you all just going in and changing the HA nodes to point at your own instances each time I update this?

EDIT: Or using the scrubber you linked to replace the id... hah

@sstratoti
Copy link
Author

And next week turned into next year...
Sorry, work/life has been kicking my ass. I'm planning some time off over the next month or two, so I'll see about publishing it then. :)

@bonanitech
Copy link

It looks like there is a problem with nodes that depend on configuration nodes in subflow modules. We'll have to wait for a fix first.

@sstratoti
Copy link
Author

You mean for the subflow module? Also, you reminded me. I updated the code with a bug fix. I'll push that up now.

@sstratoti
Copy link
Author

Allows you to pass in an msg.notificationOverride.url into the subflow.
I use this to send me an alert when a new version of HA is available, and dynamically send in the URL to the blog post. :)
I'm also doing the same for HACS updates, and I send the user to the /hacs page in the app so I can easily (and heedlessly) keep things on the bleeding edge.

@bonanitech
Copy link

You mean for the subflow module?

Yes, I built a subflow module last week, and I noticed the issue during my testings. After installing it on a different machine, it didn’t work as expected because the reference to the configuration node was not valid anymore. Unfortunately, there's currently no way to manage the link between nodes inside a subflow module and configuration nodes.

I reported this issue via Slack a few days ago, and there is a post about the same issue on the forum (link below).

https://discourse.nodered.org/t/packaging-a-subflow-as-a-module-missing-broker-configuration/55485

@henriklund
Copy link

Out of curiosity, is there a reason why this subflow is limited to four actions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment