-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature Request]: Provide option to trigger additional actions #69
Comments
There is a seperate branch called custom action. I built in months ago as many people requested this. But no-one is providing feedback on it. Please try it and let me know what can be improved for your use case |
Perfect. I'll check it out and let you know. Thanks. |
Hi, I am using the custom action branch but am unable to see the additional options. Am i missing anything? |
Please use the beta |
Any feedback @reesion ? |
Hi, unfortunately I am unable to get it to work. I added a condition for after sunset and before sunrise to trigger a light switch but it doesn’t seem to work. |
It had been a while since I created it so after reviewing the code, I have reworked it just now. You will see 3 custom action fields now with some guidance |
C
can’t seem to find the beta codes. |
https://github.com/SgtBatten/HA_blueprints/blob/custom_action/Frigate%20Camera%20Notifications/Beta |
this doesn’t seem to work. I am using the ‘Auto trigger’ and have removed the conditions to eliminate the issue and just put a service Switch: Toggle to turn on a switch but still not working. |
Okay I will try to test soon The manual trigger is working for me. Tested that yesterday to toggle a light |
I have tested the manual trigger and can confirm it worked as well. Just couldn't get the 'Auto trigger' working |
I've also been trying to get auto actions working but they are not triggering. Commented out most of the conditions leaving jus the length condition and it seems to still evaluate to false. Also tried to redefine it as a sequence variable since blueprint inputs are said to not work in template, but that also evaluated to false.
|
I haven't had a chance to try yet. When looking at the trace tree and clicking on the conditions step for this does it say condition 0 is false? |
Creating
btw this is the custom action I am trying to execute.
EDIT: Retested with the variable and its working now. |
Oh I know what it is. Edit; you were on to it. I wasn't reading the input before testing how long it was. So always false as the variable didn't exist. I'll fix it later today. |
Updated the custom action branch. I have the looped action working fine now implementing a version of your fix @krystiancharubin so thanks. |
Confirmed the new beta is working. Merging of variables in the custom action appears to work as well. |
Confirmed the new beta is working for me as well. Thank you |
I just tested this, works great and is exactly what I need. Would greatly appreciate it if this was merged back into main. Thanks for this! |
Thanks for the feedback all. Will merge when I get a minute. Hopefully not long |
Does the custom action fire after the notification is sent or before? |
Before from memory |
closed with #96 |
Looks like there's no new release created? |
The releases don't do anything, it's just a snapshot in time. |
Will this be merged to the stable branch? |
Yes I've been meaning to for a long time. I need to not merge all the beta functions but most are stable now. |
I copied it into my local stable copy and it works fine. |
Describe what you are trying to accomplish and why in non technical terms
I want to be able to trigger additional actions on run so that I can send a message to google home or turn on a light based on occupacy for example
Describe the solution you'd like
Add an extra option to trigger a service / action or even a script in addition to sending a notification
Describe alternatives you've considered
Can't think of any while being able to utilise the benefits of this blueprint configuration. Option used currently is to create a separate automation triggered by the camera occupancy status but that doesn't easily provide the same filtering and options unless they are fully specified manually.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: