Support compose mail to all users #204
Conversation
|
Subject should not ba part of the mail body. 
|
| const attachmentRules = [ | ||
| (files: File[]) => { | ||
| if (!files || files.length === 0) return true | ||
| const maxSize = 10 * 1024 * 1024 // 10MB | ||
| const oversized = files.find(file => file.size > maxSize) | ||
| if (oversized) return `File "${oversized.name}" is too large. Maximum size is 10MB.` | ||
| return true | ||
| }, | ||
| (files: File[]) => { | ||
| if (!files || files.length === 0) return true | ||
| if (files.length > 5) return 'Maximum 5 files allowed' | ||
| return true | ||
| } | ||
| ] |
There was a problem hiding this comment.
Attachment rules allow multi attachment files but this is not supported from UI. Only one attachement is enabled per mail.
frontend/kubecloud/src/utils/api.ts
Outdated
| method, | ||
| headers: requestHeaders, | ||
| body: body ? JSON.stringify(body) : undefined, | ||
| body: body instanceof FormData ? body : (body ? JSON.stringify(body) : undefined), |
There was a problem hiding this comment.
| body: body instanceof FormData ? body : (body ? JSON.stringify(body) : undefined), | |
| body: body instanceof FormData ? body : JSON.stringify(body), |
There was a problem hiding this comment.
The body is optional, this was for handling that to avoid passing an undefined body to stringify
frontend/kubecloud/src/utils/api.ts
Outdated
| method, | ||
| headers: requestHeaders, | ||
| body: body ? JSON.stringify(body) : undefined, | ||
| body: body instanceof FormData ? body : (body ? JSON.stringify(body) : undefined), |
There was a problem hiding this comment.
| body: body instanceof FormData ? body : (body ? JSON.stringify(body) : undefined), | |
| body: body instanceof FormData ? body : JSON.stringify(body), |
There was a problem hiding this comment.
The body is optional; this was for handling that to avoid passing an undefined body to stringify
There was a problem hiding this comment.
I extracted this to another variable to be reusable but the logic still the same
const bodyData = body instanceof FormData ? body : (body ? JSON.stringify(body) : undefined)
| </template> | ||
|
|
||
| <script setup lang="ts"> | ||
| import { ref, computed } from 'vue' |
There was a problem hiding this comment.
Remove unused computed prop
| const message = ref('') | ||
| const attachments = ref<File[]>([]) | ||
| const sending = ref(false) | ||
| const result = ref<{ success: boolean; message: string } | null>(null) |
There was a problem hiding this comment.
| const result = ref<{ success: boolean; message: string } | null>(null) |
Only used in catch and not needed. you can use console.error instead.
|
Please remove all unnecessary new lines. |
result email |
Description
add mail card, admin can send an email with attachments to all system users
Changes
Related Issues
Checklist