fix: multiple slack notifications (#10190)

Signed-off-by: Matt Krick <matt.krick@gmail.com>
ParabolInc · Sep 6, 2024 · c4444ef · c4444ef
1 parent 9148205
commit c4444ef
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/packages/server/dataloader/integrationAuthLoaders.ts b/packages/server/dataloader/integrationAuthLoaders.ts
@@ -187,6 +187,7 @@ export const slackNotificationsByTeamIdAndEvent = (parent: RootDataLoader) => {
       .flat()
 
     return keys.map((key) => {
+      const usedChannelIds = new Set<string>()
       return res
         .filter((doc) => doc.teamId === key.teamId && doc.event === key.event)
         .map((notification) => {
@@ -200,6 +201,11 @@ export const slackNotificationsByTeamIdAndEvent = (parent: RootDataLoader) => {
           }
         })
         .filter(isValid)
+        .filter(({channelId}) => {
+          if (!channelId || usedChannelIds.has(channelId)) return false
+          usedChannelIds.add(channelId)
+          return true
+        })
     })
   })
 }