Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mostrar error al generar email #145

Merged
merged 7 commits into from
May 20, 2024
Merged

Conversation

marcserranotouil
Copy link
Contributor

@marcserranotouil marcserranotouil commented Mar 14, 2024

Objetivos

  • Mostrar error cuando el cuerpo del correo electrónico no se puede generar desde el asistente de previsualización.

Comportamiento antiguo

  • Se seleccionaba la referencia de la plantilla con el nombre del caso del que se quiere visualizar el correo electrónico, y se rellenaban los campos del correo asociados al caso de forma automática. El problema es que no se podía generar el cuerpo del correo debido a un error. Entonces había un botón para enviar el correo y no se tractaba el error del cuerpo, lo que permitía enviar correos sin cuerpo.

Comportamiento nuevo

  • El botón "Generate Email" se ha modificado su nombre por "Send Email", y se ha añadido un nuevo botón "Generate Email". Se selecciona el caso del que se quiere ver su correo para después poder enviar-lo. Ahora para que se rellenen los campos es necesario pulsar un botón de "Generate Email", que rellena de forma automática los campos del correo asociados al caso. En el caso que no se pueda generar el cuerpo del correo debido al error, se escribe el error en el cuerpo del correo. En el caso que haya error en el cuerpo del correo, el botón de enviar correo desaparece para evitar que el correo sea enviado.

Afectaciones / Migración de datos

  • Código. Reiniciar servicios
  • Actualización módulos:
    • especificar que módulos
  • Migración de datos
    • poweremail (5.0.23.12.0)

@marcserranotouil marcserranotouil temporarily deployed to erp-ti March 14, 2024 08:17 Inactive
@marcserranotouil marcserranotouil temporarily deployed to erp-ti March 14, 2024 08:45 Inactive
@marcserranotouil marcserranotouil temporarily deployed to erp-ti March 14, 2024 09:09 Inactive
@marcserranotouil marcserranotouil self-assigned this Mar 14, 2024
@marcserranotouil marcserranotouil added this to the v23.12 milestone Mar 14, 2024
@marcserranotouil marcserranotouil temporarily deployed to erp-ti March 14, 2024 10:17 Inactive
Copy link

github-actions bot commented Mar 26, 2024

PY2TestsResults

73 tests   73 ✅  1m 13s ⏱️
 2 suites   0 💤
 2 files     0 ❌

Results for commit 34bb7e8.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Mar 26, 2024

PY3TestsResults

73 tests   73 ✅  1m 16s ⏱️
 2 suites   0 💤
 2 files     0 ❌

Results for commit 34bb7e8.

♻️ This comment has been updated with latest results.

@lcbautista lcbautista merged commit b10b526 into v5_backport May 20, 2024
5 of 7 checks passed
@lcbautista lcbautista deleted the 56615_mostrar_error_email branch May 20, 2024 07:53
@roniFCA roniFCA deployed to benasque-comer-erp June 12, 2024 07:47 Active
@lcbautista lcbautista temporarily deployed to frankenergie-erp July 17, 2024 15:52 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants