Conflict between `alert_once` and `group` options for notifications

Home Assistant Android version: 2021.9.0

Android version: 11

Phone model: Pixel 5

Home Assistant version: 2021.10.0

Description of problem: The alert_once option for notifications does not work properly if a group is also specified.

Additional information:

Easily repeatable using developer tools to call the service:

service: notify.mobile_app_pixel_5
data:
  message: aoeu {{range(1,99)|random}}
  data:
    # group: "aoeu"
    tag: blah
    alert_once: true

When uncommenting the group line, a sound will be played every time the notification is sent. When it’s commented out, alert_once works properly and only plays a sound if the notification isn’t already visible.

[dshokouhi]

Ah ha I found the issue as to why it did not work earlier, for groups we have an additional notify call to post the notification in order to group them and what happened was the group data that I tested with was not there.

For this we need to set GROUP_ALERT_CHILDREN as the group alert behavior because the first notification posted to the group is the group summary. Will get a PR ready to fix 😃

[gazpachoking]

Doing some initial testing and none of those optoins silence a notification in the group, I think we may just want to tell users not to group these notifications

Man, that’s disappointing. Perhaps a bug in android itself then. Really I see no reason or documentation that the onlyAlertOnce shouldn’t work in groups. I was hoping the groupAlert behavior could fix that deficiency, but 🤷‍♂️