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

Plantilla genérica de correo electrónico #38

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

lcbautista
Copy link
Member

@lcbautista lcbautista commented Jul 16, 2024

Objetivos

Facilitar la creación y modificación de plantillas de correo electrónico

Comportamiento antiguo

Actualmente para crear una plantilla de correo se tiene que "crear" (o hacer un copy-paste de una plantilla existente) de cero todo el cuerpo del correo.

Esto no es eficiente ya que si el estilo del correo (branding) es común para todas las comunicaciones, el día que se quiera realizar alguna modificación se tendrá que ir plantilla por plantilla realizando las modificaciones pertinentes.

Si en cambio se necesita crear una nueva comunicación, se recurrirá al copy-paste el cual puede dar lugar a error.

Comportamiento nuevo

Este módulo implementa una plantilla genérica que permite ser fácilmente editable utilizando los banners y los dominios de banners para tener un contenido personalizable por plantilla.

La plantilla genérica esta constituida por 3 banners que son visibles en el contenido del correo:

  • generic_email_template_header
  • generic_email_template_body
  • generic_email_template_footer

imagen

y otros dos que corresponden a "metadatos" del propio correo:

  • generic_email_template_title: se utiliza como nombre del HTML generado. No se suele mostrar en los correos pero se ha creado para mantener el estándar HTML.
  • generic_email_template_preheader: algunos clientes de correo utilizan este tag como un asunto que se visualiza desde el listado de correos. Vendría a ser un "asunto resumido"

Un ejemplo de como se puede utilizar esta plantilla la encontraremos en la siguiente pull request: https://github.com/gisce/erp/pull/21041

@lcbautista lcbautista self-assigned this Jul 16, 2024
@lcbautista lcbautista changed the title new: module with generic email template Plantilla genérica de correo electrónico Jul 16, 2024
@lcbautista lcbautista force-pushed the new_generic_email_template branch from d07417c to 2547459 Compare July 17, 2024 10:07
@lcbautista lcbautista force-pushed the new_generic_email_template branch from 2547459 to 7b9163e Compare July 17, 2024 10:29
@lcbautista lcbautista deployed to AbeERP_PRE July 17, 2024 10:43 Active
@lcbautista lcbautista temporarily deployed to frankenergie-pre July 17, 2024 12:04 Inactive
@lcbautista lcbautista deployed to frankenergie-erp July 17, 2024 16:04 Active
@lcbautista lcbautista added the deployed Pull request deployed label Jul 17, 2024
@polsala polsala added the to be merged Ready to integrate label Jul 17, 2024
@polsala polsala enabled auto-merge July 17, 2024 20:31
@polsala polsala disabled auto-merge July 17, 2024 20:36
Copy link

Test Results

70 tests   70 ✅  0s ⏱️
 2 suites   0 💤
 2 files     0 ❌

Results for commit 7b9163e.

@polsala
Copy link
Member

polsala commented Jul 17, 2024

Legalmente paguen a este repositorio por las extracciones ilegales que se an cometido 8952140061899836446 sipei o stp

¿en Bitcoin o Dogecoin?

image

@lcbautista lcbautista merged commit a0e7c7d into master Jul 18, 2024
5 checks passed
@lcbautista lcbautista deleted the new_generic_email_template branch July 18, 2024 07:02
@davidrabassa davidrabassa temporarily deployed to gisceerppre July 22, 2024 08:06 Inactive
@AdriaGual AdriaGual temporarily deployed to gasiluz-pre July 23, 2024 07:35 Inactive
@davidrabassa davidrabassa deployed to energetica-pre July 24, 2024 12:40 Active
@roniFCA roniFCA requested a deployment to cerp-pre July 31, 2024 08:01 Pending
@roniFCA roniFCA requested a deployment to cerp-pre July 31, 2024 08:03 Pending
@lcbautista lcbautista temporarily deployed to gisceerpprod August 8, 2024 12:01 Inactive
@roniFCA roniFCA requested a deployment to dyneff-pre August 12, 2024 08:25 Pending
@roniFCA roniFCA requested a deployment to dyneff-pre August 12, 2024 08:32 Pending
@roniFCA roniFCA requested a deployment to dyneff-pre August 12, 2024 08:39 Pending
@roniFCA roniFCA requested a deployment to dyneff-pre August 12, 2024 08:41 Pending
@roniFCA roniFCA requested a deployment to dyneff-pre August 12, 2024 08:45 Pending
@roniFCA roniFCA requested a deployment to dyneff-pre August 12, 2024 08:55 Pending
@lcbautista lcbautista temporarily deployed to ecasa-pre August 19, 2024 14:19 Inactive
@joanperez1 joanperez1 temporarily deployed to opcionenergia-pre August 27, 2024 12:33 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed PRE deployed Pull request deployed to be merged Ready to integrate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants