search

nextcloud / notifications

:bell: Notifications app for Nextcloud
GNU Affero General Public License v3.0
114 stars 54 forks source link

fix(browser): Don't create web notifications for old notifications #1943

Closed nickvergessen closed 3 weeks ago

nickvergessen commented 3 weeks ago

Sometimes a notification got "newly mounted" while being old. This can happen when a user has many notifications (100-1). The UI first only loads (100-76), if any notification is then resolved (e.g. by deleting or reading a chat), further old notifications (75+74) would be added to the UI and triggered a web notification (including call sound) in the past.

This threshold ID is therefore updated to only higher values, before each pulling of notifications to ensure that we only ever web-notify on new notifications and not newly loaded old notifications.

nickvergessen commented 3 weeks ago

/compile /

nickvergessen commented 3 weeks ago

/backport 6a543679b4de8af65cc82baa233ae17ffbbbc1af to stable29

nickvergessen commented 3 weeks ago

/backport 6a543679b4de8af65cc82baa233ae17ffbbbc1af to stable28

nickvergessen commented 3 weeks ago

/backport 6a543679b4de8af65cc82baa233ae17ffbbbc1af to stable27