FIX ensure notifications are not sent back to the author of the messa…

…ge, unless they mention themselves
guardian · Aug 9, 2022 · a5d57d5 · a5d57d5
1 parent 7bb9161
commit a5d57d5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/notifications-lambda/src/index.ts b/notifications-lambda/src/index.ts
@@ -75,7 +75,8 @@ export const handler = async (event: DynamoDBStreamEvent) => {
               // TODO: Include more scenarios that trigger desktop notification
               (item) =>
                 isUserMentioned(item, user) ||
-                doesUserManuallyHavePinboardOpen(item, user)
+                (item.userEmail !== user.email && // ensure we don't notify the person who sent the message
+                  doesUserManuallyHavePinboardOpen(item, user))
             )
             .map((item) =>
               webPush