From 79ffec50df62b301855592aae59191c4baf2df38 Mon Sep 17 00:00:00 2001 From: Ferruh Cihan <63190600+ferruhcihan@users.noreply.github.com> Date: Fri, 29 May 2026 00:16:14 +0200 Subject: [PATCH] fix: update notification receivers logic in teams --- src/pages/teams/create-edit/TeamsCreateEditPage.tsx | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/pages/teams/create-edit/TeamsCreateEditPage.tsx b/src/pages/teams/create-edit/TeamsCreateEditPage.tsx index 4ff6d9f3..4e819b59 100644 --- a/src/pages/teams/create-edit/TeamsCreateEditPage.tsx +++ b/src/pages/teams/create-edit/TeamsCreateEditPage.tsx @@ -139,10 +139,11 @@ export default function TeamsCreateEditPage({ * alertManager needs to be configured with 'receivers: null' if there are no recievers and 'none' is currently * the way to configure that. */ - const rawReceivers = submitData.spec?.alerts?.receivers ?? [] - let receivers = rawReceivers.filter((r) => r !== 'none') + let receivers: ('slack' | 'msteams' | 'none')[] = [] + if (submitData.spec?.alerts?.slack?.url) receivers.push('slack') + if (submitData.spec?.alerts?.msteams?.lowPrio || submitData.spec?.alerts?.msteams?.highPrio) + receivers.push('msteams') - // @ts-ignore: no receivers requires 'none' string if (submitData.spec?.managedMonitoring?.alertmanager && receivers.length === 0) receivers = ['none'] // 3) Combine edge cases with submittedData for final payload