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

Enhancement - SinergiaDA - Varios cambios #421

Merged
merged 8 commits into from
Oct 24, 2024
Merged

Conversation

juanSTIC
Copy link
Collaborator

@juanSTIC juanSTIC commented Oct 4, 2024

1- Cambiar prefijo SDA_ a SCRM_

Para mejorar la consistencia en los terminos utilizados y para facilitar la comunicación y comprensión de documentación, se cambia el prefijo SDA_ por SCRM_ en los nombres d elos grupos que proceden de SinergiaCRM.

Como probarlo

  • Por inspección de código.
  • Verificando que tras la reconstrucción, los grupos de SinergiaCRM han tomado el prefijo SCRM_

2- Cambiar tipo de datos para campos numéricos

Hasta el momento el contenido de todos los campos en las vistas sda_ se guardaban con el tipo varchar, sin embargo se ha visto que las columnas de tipo numérico, deben crearse con un tipo numérico específico que permita que la ordenación de haga correctamente en estos casos, puesto que en los casos en que se usa min y max el resultado puede variar considerablemente.
Como probarlo

  • Verificar que tras la reconstrucción los campos de tipo numérico del CRM (decimal, int, currency y float) son convertidos al tipo decimal (10,4)
  • Verificar que en la instancia de SinergiaDA, las agregaciones MIN y MAX funcionan correctamente sobre campos lo campos numéricos

3- Omitir administradores de la lista de permisos de sda_def_permissions

Se ha comprobado que no es necesario incluir permisos específicamente para los usuarios administradores, ya que estos tienen todos los permisos posibles por su pertenencia al grupo EDA_ADMIN, al que se les añade previamente en el script de rebuild.
Aquí se omiten los permisos para cualquier usuario administrador y con ello se consigue una mayor limpieza de a interfaz de administración de SinergiaDA.

Cómo probarlo:

  • Verificar que tras rebuild, no se han añadido permisos de ningún usuario administrador en la vista sda_def_permissions

@juanSTIC juanSTIC self-assigned this Oct 4, 2024
@juanSTIC juanSTIC added enhancement New feature or request SinergiaDA labels Oct 4, 2024
Copy link

github-actions bot commented Oct 4, 2024

Actions executed at: 2024-10-24 08:19:28.

@juanSTIC juanSTIC changed the title Enhancement - SinergiaDA - Cambiar prefijo SDA_ a SCRM_ Enhancement - SinergiaDA - Varios cambios Oct 4, 2024
Copy link
Collaborator

@PaulaaSTIC PaulaaSTIC 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

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

@PaulaaSTIC PaulaaSTIC merged commit 3b45bb9 into develop Oct 24, 2024
1 check passed
@PaulaaSTIC PaulaaSTIC deleted the enhancement/SDAOctober branch October 24, 2024 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request SinergiaDA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants