verdnatura/salix
verdnatura
/
salix
3
5
Fork 0
Code Issues Pull Requests 66 Releases Wiki Activity

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
Conversation 0 Commits 2 Files Changed 1 +39 -12
1 changed files with 39 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

51
back/methods/osticket/sendToSupport.js
View File

@ -33,25 +33,52 @@ module.exports = Self => {
const emailUser =
await Self.app.models.EmailUser.findById(userId, {fields: ['email']});
let html = `<h2>Motivo: ${reason}</h2>`;
html += `<h3>Usuario: ${userId} ${emailUser.email}</h3>`;
html += `<h3>Additional Data:</h3>`;
html += '<ul>';
const tableStyle = 'width:100%; border-collapse: collapse; text-align: left;';
jorgep commented 2025-02-12 13:20:37 +00:00
Review
Copy Link

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.
const thStyle = 'padding: 8px; border: 1px solid #ddd; background-color: #f4f4f4;';
const tdStyle = 'padding: 8px; border: 1px solid #ddd;';
const tdBoldStyle = 'padding: 8px; border: 1px solid #ddd; font-weight: bold;';
const subTdStyle = 'padding: 6px; border: 1px solid #ddd;';
const subTdBoldStyle = 'padding: 6px; border: 1px solid #ddd; font-weight: bold;';
let html = `
<h2>Motivo: ${reason}</h2>
<h3>Usuario: ${userId} ${emailUser.email}</h3>
<h3>Additional Data:</h3>
<table style="${tableStyle}">
<thead>
<tr>
<th style="${thStyle}">Clave</th><th style="${thStyle}">Valor</th></tr>
</thead>
<tbody>`;
for (const [key, val] of Object.entries(additionalData)) {
if (key !== 'config') html += `<li>${key}: ${parse(val)}</li>`;
else {
html += `<li>${key}:</li><ul style="list-style-type: square;">`;
for (const [confKey, confVal] of Object.entries(val))
html += `<li>${confKey}: ${parse(confVal)}</li>`;
html += '</ul>';
if (key !== 'config') {
html += `<tr>
<td style="${tdBoldStyle}">${key}</td>
<td style="${tdStyle}">${parse(val)}</td>
</tr>`;
} else {
html += `<tr>
<td style="${tdBoldStyle}">${key}</td>
<td style="${tdStyle}">
<table style="${tableStyle}">
<tbody>`;
for (const [confKey, confVal] of Object.entries(val)) {
html += `<tr>
<td style="${subTdBoldStyle}">${confKey}</td>
<td style="${subTdStyle}">${parse(confVal)}</td>
</tr>`;
}
html += `</tbody></table></td></tr>`;
}
}
html += '</ul>';
html += `</tbody></table>`;
const {message, path, name} = additionalData;
const err = name && message ? `${name}: ${message}` : name || message || '';
jorgep commented 2025-02-12 13:22:13 +00:00
Review
Copy Link

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

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

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) .
html
});
};