Merge pull request #149 from guardian/fix/dont-notify-own-messages

FIX ensure notifications are not sent back to the author of the message
guardian · Aug 10, 2022 · ce8309a · ce8309a
2 parents 4f79fae + c034635
commit ce8309a
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