Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Skip to content
Commit b479d648 authored by Jeff DeCew's avatar Jeff DeCew Committed by Android Build Coastguard Worker
Browse files

Allow alertOverride if the summary GROUP_ALERT_ALL

Because we still require all children to have GROUP_ALERT_PARENT, the meaning of GROUP_ALERT_ALL is equivalent to GROUP_ALERT_PARENT in any case where we would have an alertOverride, so we now allow either of those values.
Also of note, because the alertOverride absolutely needs to have GROUP_ALERT_PARENT, and the semantics of heterogeneous child alert behaviors is unclear, we're keeping the requirement that all children have GROUP_APERT_PARENT to avoid over-applying this logic.

Bug: 185680162
Test: atest NotificationGroupManagerLegacyTest
Merged-In: I63a51cc1d94fc8a8617ca76a0fa479e3829816c6
Change-Id: I63a51cc1d94fc8a8617ca76a0fa479e3829816c6
(cherry picked from commit 6ee64f6f)
(cherry picked from commit 99b1011e)
Merged-In:I63a51cc1d94fc8a8617ca76a0fa479e3829816c6
parent 878e9322
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment