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

Hotfix - General - Usar el delimitador de decimales configurado en campos FLOAT #338

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

ManuSinergiaCRM
Copy link
Collaborator

@ManuSinergiaCRM ManuSinergiaCRM commented Aug 14, 2024

Para los casos relacionados con Plantillas de Email y PDF

Queda pendiente crear el PR to SA del caso detectado en el proceso de importación

Descripción

Se detecta que el CRM no usa correctamente el delimitador de decimales cuando opera con campos de tipo FLOAT en los siguientes casos de uso:

  1. Al importar un campo de tipo Float
  2. Al usar una plantilla de correo desde campañas o desde un flujo de trabajo
  3. Al exportar una plantilla PDF

Pruebas

Este PR corrige que no se transforme correctamente el símbolo para separar decimales de un valor decimal obtenido de base de datos al incorporarlo a una plantilla de correo o una plantilla PDF.

Pruebas

  1. Crear un campo de tipo Float en el módulo de Personas

Plantillas de correo
2. Crear una plantilla de correo con ese campo.
3. Crear una persona y proveer un valor en ese campo
4. Crear una campaña y un email marketing que use esa plantilla
5. Enviar el email, comprobar que el valor del campo Float es el mismo y que usa el símbolo decimal configurado en el CRM

Flujos de trabajo (opcional, ya que no se ha corregido nada)
6. Crear un FdT cuya acción sea enviar un email y que use la plantilla creada en el paso 2.
7. Modificar la persona para que se envíe el email, comprobar que el valor del campo Float es el mismo y que usa el símbolo decimal configurado en el CRM

Plantillas PDF
8. Crear una plantilla PDF y añadir un campo de tipo decimal. Por ejemplo: Total annual donations.
9. Generar la plantilla para la persona usada anteriormente, comprobar que el valor del campo Float es el mismo y que usa el símbolo decimal configurado en el perfil del usuario logueado

Proceso de Importación
10. Exportar la Persona y editar el fichero generado para dejar solo la columna apellido (campo requerido) y la columna del campo Float. Cambiar el apellido para que cuando se importe se diferencie del otro registro.
11. Realizar la importación y comprobar que el valor del campo Float es el mismo y que usa el símbolo decimal configurado en el perfil del usuario logueado

Para todos los casos de uso
12. Cambiar el símbolo decimal del sistema y del usuario y volver a realizar las pruebas.

Copy link

github-actions bot commented Aug 14, 2024

Actions executed at: 2024-09-20 15:44:11.

Copy link
Collaborator

@AlbertoSTIC AlbertoSTIC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aprobado

Copy link
Collaborator

@ainaraRT ainaraRT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(A)Probado

@AlbertoSTIC AlbertoSTIC merged commit 2e54ff0 into develop Sep 20, 2024
1 check passed
@AlbertoSTIC AlbertoSTIC deleted the bug/decimalSymbolParserToFloatFields branch September 20, 2024 15:44
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.

3 participants