From 55f2a6e0007e3eec66c7eacb95ceba513d419cf6 Mon Sep 17 00:00:00 2001
From: Jakob Ackermann <jakob.ackermann@overleaf.com>
Date: Mon, 17 May 2021 11:37:32 +0200
Subject: [PATCH] Merge pull request #4017 from
 overleaf/jpa-fix-notification-ack

[NotificationsBuilder] fix too broad acknowledge API calls

GitOrigin-RevId: 4ddaeb33ccf4d4a4f5c36671c720ec3cd32ef761
---
 .../app/src/Features/Notifications/NotificationsBuilder.js    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/services/web/app/src/Features/Notifications/NotificationsBuilder.js b/services/web/app/src/Features/Notifications/NotificationsBuilder.js
index 950e37f8dd..33d25d0fe7 100644
--- a/services/web/app/src/Features/Notifications/NotificationsBuilder.js
+++ b/services/web/app/src/Features/Notifications/NotificationsBuilder.js
@@ -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)
     },
   }
 }