Skip to content

Commit 09cb148

Browse files
FIX (notifications): Get rid of password validation for email
1 parent b6518ef commit 09cb148

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed
Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import type { EmailNotifier } from './EmailNotifier';
22

3-
export const validateEmailNotifier = (isCreate: boolean, notifier: EmailNotifier): boolean => {
3+
export const validateEmailNotifier = (notifier: EmailNotifier): boolean => {
44
if (!notifier.targetEmail) {
55
return false;
66
}
@@ -13,9 +13,5 @@ export const validateEmailNotifier = (isCreate: boolean, notifier: EmailNotifier
1313
return false;
1414
}
1515

16-
if (isCreate && !notifier.smtpPassword) {
17-
return false;
18-
}
19-
2016
return true;
2117
};

frontend/src/features/notifiers/ui/edit/EditNotifierComponent.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@ export function EditNotifierComponent({
182182
}
183183

184184
if (notifier.notifierType === NotifierType.EMAIL && notifier.emailNotifier) {
185-
return validateEmailNotifier(!notifier.id, notifier.emailNotifier);
185+
return validateEmailNotifier(notifier.emailNotifier);
186186
}
187187

188188
if (notifier.notifierType === NotifierType.WEBHOOK && notifier.webhookNotifier) {

0 commit comments

Comments
 (0)