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

Improvement - Personas y Organizaciones - Incluir índice por número de identificación #412

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

enricsinergia
Copy link
Collaborator

Descripción

Tal como se explica en #392 se detecta que los formularios hacen uso intensivo de la búsqueda por número de identificación para evitar duplicar personas/organizaciones en el CRM. Esa búsuqeda no tiene índice que la facilite por lo que en BBDD grandes y cuando hay aluvión de peticiones puede colapsar el sistema.

Solución

Incluir los índices por número de identificación en Personas y Organizaciones. No se incluye en Interesados puesto que los formularios de interesados no realizan ese chequeo.
Los índices se incluyen vía SQL al no permitir SuiteCRM definir índices sobre campos custom mediante vardefs.

Pruebas

1.- Ejecutar el fichero SQL de creación de índices
2.- Conectarse a la BBDD y comrpobar que los índices sobre el campo stic_identification_number_c se han creados en contacts_cstm y accounts_cstm

Copy link

github-actions bot commented Oct 1, 2024

Actions executed at: 2024-10-01 04:38:51.

Copy link
Collaborator

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

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants