Do not notify again for old events
Resending the notification here can trigger other system components or apps that listen to new notifications, such as connected smart watches or automation tools. Hopefully fixes https://github.com/SchildiChat/SchildiChat-android/issues/11 Change-Id: I9fe77f8d32c802a14439179b054727733bbe1cf8
This commit is contained in:
parent
d13891d556
commit
6b7564f9f4
|
@ -466,7 +466,7 @@ class NotificationDrawerManager @Inject constructor(private val context: Context
|
||||||
|
|
||||||
if (eventList.isEmpty() || eventList.all { it.isRedacted }) {
|
if (eventList.isEmpty() || eventList.all { it.isRedacted }) {
|
||||||
notificationUtils.cancelNotificationMessage(null, SUMMARY_NOTIFICATION_ID)
|
notificationUtils.cancelNotificationMessage(null, SUMMARY_NOTIFICATION_ID)
|
||||||
} else {
|
} else if (hasNewEvent) {
|
||||||
// FIXME roomIdToEventMap.size is not correct, this is the number of rooms
|
// FIXME roomIdToEventMap.size is not correct, this is the number of rooms
|
||||||
val nbEvents = roomIdToEventMap.size + simpleEvents.size
|
val nbEvents = roomIdToEventMap.size + simpleEvents.size
|
||||||
val sumTitle = stringProvider.getQuantityString(R.plurals.notification_compat_summary_title, nbEvents, nbEvents)
|
val sumTitle = stringProvider.getQuantityString(R.plurals.notification_compat_summary_title, nbEvents, nbEvents)
|
||||||
|
|
Loading…
Reference in New Issue