feat: refs #8571 enhance email formatting in sendToSupport function with structured HTML table #3445

Merged
jorgep merged 2 commits from 8571-hotfix-notShowToken into master 2025-02-13 11:01:38 +00:00
Member
No description provided.
jorgep added 1 commit 2025-02-12 13:16:29 +00:00
jorgep reviewed 2025-02-12 13:20:37 +00:00
@ -37,3 +36,1 @@
html += `<h3>Usuario: ${userId} ${emailUser.email}</h3>`;
html += `<h3>Additional Data:</h3>`;
html += '<ul>';
const tableStyle = 'width:100%; border-collapse: collapse; text-align: left;';
Author
Member

ChatGPT 15-20 min. Resultado en un comentario de la tarea. He aprovechado para cambiarlo porque queda bastante más legible y era algo rápido.

ChatGPT 15-20 min. Resultado en un comentario de la tarea. He aprovechado para cambiarlo porque queda bastante más legible y era algo rápido.
jorgep reviewed 2025-02-12 13:22:13 +00:00
@ -50,2 +74,4 @@
html += `</tbody></table>`;
const {message, path, name} = additionalData;
const err = name && message ? `${name}: ${message}` : name || message || '';
Author
Member

Así, no pondrá nunca undefined: undefined o undefined: mi mensaje de error...

Así, no pondrá nunca undefined: undefined o undefined: mi mensaje de error...
jorgep reviewed 2025-02-12 13:23:48 +00:00
@ -52,3 +79,3 @@
await smtp.send({
to: `${config.app.reportEmail}, ${emailUser.email}`,
subject: `[Support-Salix] ${path} ${name}: ${message}`,
subject: `[Support-Salix] ${path.split('?')[0]} ${err}`,
Author
Member

Tras lo hablado con Satorres, hemos quedado que los query-params molestan en el asunto del cau, en el mensaje se pueden ver pero en el asunto solo se ve el path (#/item/fixed-price) .

Tras lo hablado con Satorres, hemos quedado que los query-params molestan en el asunto del cau, en el mensaje se pueden ver pero en el asunto solo se ve el path (#/item/fixed-price) .
jorgep requested review from alexm 2025-02-12 13:25:38 +00:00
alexm approved these changes 2025-02-13 06:41:33 +00:00
jorgep added 1 commit 2025-02-13 10:59:43 +00:00
jorgep merged commit e99fbca255 into master 2025-02-13 11:01:38 +00:00
jorgep deleted branch 8571-hotfix-notShowToken 2025-02-13 11:01:38 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix#3445
No description provided.