github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2025-04-20 16:25:43 +00:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #4017 from overleaf/jpa-fix-notification-ack

Browse source 
[NotificationsBuilder] fix too broad acknowledge API calls

GitOrigin-RevId: 4ddaeb33ccf4d4a4f5c36671c720ec3cd32ef761
This commit is contained in:
Jakob Ackermann 2021-05-17 11:37:32 +02:00 • committed by Copybot
parent 1d95137e13
commit 55f2a6e000
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
services/web/app/src/Features/Notifications/NotificationsBuilder.js
View file

@ -51,7 +51,7 @@ function featuresUpgradedByAffiliation(affiliation, user) {
if (callback == null) {
callback = function () {}
}
NotificationsHandler.markAsReadByKeyOnly(this.key, callback)
NotificationsHandler.markAsReadWithKey(user._id, this.key, callback)
},
}
}
@ -78,7 +78,7 @@ function redundantPersonalSubscription(affiliation, user) {
if (callback == null) {
callback = function () {}
}
NotificationsHandler.markAsReadByKeyOnly(this.key, callback)
NotificationsHandler.markAsReadWithKey(user._id, this.key, callback)
},
}
}