Simplify null checks in sendEmail validation logic

This commit is contained in:
2025-09-13 21:06:54 +02:00
parent e1800f5a6e
commit b342f32d97

View File

@@ -79,7 +79,7 @@ export const sendEmail = async <T extends BaseEmailData = BaseEmailData>(
throw new Error('Field "to" is required for sending emails')
}
if (!emailData.subject || emailData.subject === null || !emailData.html || emailData.html === null) {
if (!emailData.subject || !emailData.html) {
throw new Error('Fields "subject" and "html" are required when not using a template')
}
@@ -87,10 +87,10 @@ export const sendEmail = async <T extends BaseEmailData = BaseEmailData>(
if (emailData.to) {
emailData.to = parseAndValidateEmails(emailData.to as string | string[])
}
if (emailData.cc && emailData.cc !== null) {
if (emailData.cc) {
emailData.cc = parseAndValidateEmails(emailData.cc as string | string[])
}
if (emailData.bcc && emailData.bcc !== null) {
if (emailData.bcc) {
emailData.bcc = parseAndValidateEmails(emailData.bcc as string | string[])
}
@@ -108,4 +108,4 @@ export const sendEmail = async <T extends BaseEmailData = BaseEmailData>(
return email as unknown as T
}
export default sendEmail
export default sendEmail