fix: send only summary for large slack notifications (wundergraph#556)

pedraumcosta · Feb 19, 2024 · 732dbc5 · 732dbc5
1 parent c56ce3c
commit 732dbc5
Showing 1 changed file with 17 additions and 0 deletions.
diff --git a/controlplane/src/core/webhooks/OrganizationWebhookService.ts b/controlplane/src/core/webhooks/OrganizationWebhookService.ts
@@ -165,6 +165,23 @@ export class OrganizationWebhookService {
         (c) => c.changeType.includes('ADDED') || c.changeType.includes('CHANGED'),
       );
       const removedChanges = latestChangelogs.changelogs.filter((c) => c.changeType.includes('REMOVED'));
+
+      if (removedChanges.length + addedChanges.length > 20) {
+        tempData.attachments.unshift({
+          color: '#e11d48',
+          blocks: [
+            {
+              type: 'section',
+              text: {
+                type: 'mrkdwn',
+                text: `Too many changes to display. There were ${removedChanges.length} deletions and ${addedChanges.length} additions.`,
+              },
+            },
+          ],
+        });
+        return tempData;
+      }
+
       if (removedChanges.length > 0) {
         tempData.attachments.unshift({
           color: '#e11d48',