domoticz / domoticz-android

Domoticz Client Application for Android Devices
104 stars 69 forks source link

Actionnable Notifications #708

Open rezzalex opened 5 months ago

rezzalex commented 5 months ago

Like the tiles on wearOS, HA seems ahead again :

https://companion.home-assistant.io/docs/notifications/actionable-notifications

rezzalex commented 1 month ago

@galadril , I supposed it would require some common dev with the DZ project, but the result would be awesome... @gizmocuz , what do you think ?

gizmocuz commented 1 month ago

ahead 'again'? It's a (new) feature from HASS and it's indeed a nice feature for some notifications! But you could also program this

rezzalex commented 1 month ago

@gizmocuz , All my apologies if my way of expressing hurt.

Thank you for all the effort you make and the time invested in our wonderful home automation system.

I did not know that this feature was new on HA side. My remark was in comparison with the wear OS tiles, where it has exist in HA before it exist in the DZ mobile app.

I would never have the knowledge to develop this kind of functionality, and I guess the developments should be in conjunction with a mobile app. Would it be possible to imagine ranking of the most expected new features and see if that one could be welecomed by the user community ?

gizmocuz commented 1 month ago

No worries, it's probably a language thing 😁 (Or some people use the word 'again' a lot)

This you have to ask @galadril , as he's the developer of the application

rezzalex commented 1 month ago

so you imagine a configuration of this kind of notification directly from a mobile app ? not from DZ UI itself ?

gizmocuz commented 1 month ago

This is something for the mobile application and that is probably the reason why you added this request here. A user is also not constantly monitoring a desktop/tablet but does read a mobile notification

rezzalex commented 1 month ago

@galadril , what do you think?

Already feasible with tasker plugin ? An easy configuration would be the greatest feature.

galadril commented 1 month ago

To be honest, I don't really see how I could implement this in the app, in the current setup of Domoticz. I've made a quick logical split of my thoughts:

  1. One device triggers actions on another device For example, if you have motion sensors and/or switches in your home, you can configure some kind of notification for turning them on and off. How and where should the logic be implemented? What kind of buttons/actions do you have on each device?

actions-a11b1419f6254c8b98270942e29e327e

  1. One device triggers actions on itself Consider a scenario where one device triggers its own notifications and actions. If you turn on a lamp and receive a notification with buttons to turn the lamp on and off (since it’s a switch), what's the actual benefit of having those buttons on the notification, given that you just turned on that light/switch?

Is this actual handy? light 1 turned on, actions turn on/off.. probably not

  1. Notifications from scripts (like Blockly) If you have events/logic/scripts that send notifications, there is currently no possibility to send device/idx/type information to the mobile app. This makes it even harder to add any kind of logic.

image

So, I don't really see a workable implementation from the app only at the moment.

galadril commented 1 month ago

What could be possible @gizmocuz if in Domoticz for example we would adjust the notification settings page to setup actions on FCM notification. You register an action on a device, which then gets stored in Domoticz.. When you then trigger a notifications you can actually send the registerd action id's / texts through the fcm notification json to the mobile app.

image

I then show the button on the notification and when somebody clicks them, i call some kind of actions ips with the idx of the device and the action id.

galadril commented 1 month ago

But i guess this topic is more important, else we wont have notifications at all soon πŸ‘πŸΌ @gizmocuz https://github.com/domoticz/domoticz/issues/5921