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

Feature - Administración del CRM - Creación módulo Monitorización #211

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

Conversation

ainaraRT
Copy link
Collaborator

@ainaraRT ainaraRT commented Apr 24, 2024

Descripción

Este PR implementa la monitorización de los usuarios que navegan por el CRM, teniendo presente el módulo ya existente de SuiteCRM. Solamente será visible para aquellos usuarios que sean administradores.

Para ello se desarrollan los siguientes puntos:

  • Obtención de los inicios de sesión de los usuarios como logouts.
  • Muestra cuando el usuario se ha metido en la vista detalle del registro, al igual que si crea uno nuevo, lo modifica o lo borra.

Uso de la funcionalidad

  • Para poder visualizar la lista, hay que ir a Administración y en el apartado de Admin Tools, ver que existe el módulo Monitorizaciones.
  • Se visualizará una lista con todos los registros anteriores, además de los nuevos que se irán actualizando.

Puntos pendientes

  • Añadir el link que lleve a dicho registro, en el “Resumen del Ítem”.
  • Recoger los registros ya existentes y añadir el Usuario asignado.
  • Mostrar el cierre de sesión al quitar la pestaña y la sesión expirada.
  • Vista de lista/Estudio (solamente recoge los campos creados)/Cargador de módulos
  • Crear una lista donde el administrador decida qué módulos quiere que se muestren en Monitorización: Feature - Administración del CRM - Creación módulo Monitorización #211 (comment)

Pruebas

  1. Comprobar que se visualiza el módulo dentro de Administración -> Admin Tools.
  2. Comprobar en la lista que no hay ningún registro vacío, al igual que los campos. (Menos los registros antiguos que se mostrarán con el Usuario asignado vacío)
  3. Con registros de módulos diferentes, visualizar en la tabla que se añaden correctamente con las etiquetas correspondientes:
  • Registro creado. Esta acción se muestra cuando se ha creado un registro nuevo.
  • Registro modificado. Esta acción se muestra cuando se ha modificado un registro.
  • Registro eliminado. Esta acción se muestra cuando se ha eliminado un registro.
  1. Meterse dentro de un registro y comprobar que aparece cómo “Registro accedido”.
  2. Iniciar sesión con usuarios diferentes y cerrar sesión.
  3. Comprobar que no se puede editar ningún registro de la vista de lista ni se puede actualizar masivamente ni eliminar.

Copy link

github-actions bot commented Apr 24, 2024

Actions executed at: 2025-01-14 12:41:40.

@jalbaiges jalbaiges changed the title Feature - Monitorización - Creación módulo Monitorización Feature - Administración del CRM - Creación módulo Monitorización May 15, 2024
@ainaraRT ainaraRT marked this pull request as ready for review August 14, 2024 15:05
@AlbertoSTIC AlbertoSTIC requested review from ManuSinergiaCRM and jordiSTIC and removed request for AlbertoSTIC November 15, 2024 11:32
@AlbertoSTIC AlbertoSTIC dismissed their stale review November 15, 2024 11:33

Not needed

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.

Ver comentarios en código

include/MVC/View/SugarView.php Show resolved Hide resolved
data/SugarBean.php Outdated Show resolved Hide resolved
modules/Trackers/metadata/listviewdefs.php Show resolved Hide resolved
modules/Trackers/metadata/searchdefs.php Outdated Show resolved Hide resolved
modules/Trackers/monitor/Monitor.php Show resolved Hide resolved
@ManuSinergiaCRM
Copy link
Collaborator

A nivel funcional, si eliminamos un registro del módulo de Monitorización, se crea otro registro indicando que se ha eliminado un registro..

Entiendo que o bien quitamos la opción de Eliminar o hacemos que eliminar un registro de este módulo no genere otro

@ManuSinergiaCRM
Copy link
Collaborator

Al crear un registro de un módulo, ya sea al crear o al duplicar, aparecen dos registros en el módulo de Monitorización de los cuales uno no es entendible

Mensaje ilegible tras duplicar

@ManuSinergiaCRM
Copy link
Collaborator

He leído un comentario de Alberto donde he interpretado que queremos que el módulo aparezca en Estudio, sin embargo, este no aparece.

@ManuSinergiaCRM
Copy link
Collaborator

Por último, el número de registros de este módulo va a crecer de forma muy rápida pudiendo dar problemas de rendimiento como en los registros auditados de flujos de trabajo

¿Se ha valorado crear alguna automatización para que se eliminen los registros de monitorización a nivel lógico pasado un tiempo?

@ainaraRT
Copy link
Collaborator Author

ainaraRT commented Jan 9, 2025

A nivel funcional, si eliminamos un registro del módulo de Monitorización, se crea otro registro indicando que se ha eliminado un registro..

Entiendo que o bien quitamos la opción de Eliminar o hacemos que eliminar un registro de este módulo no genere otro

Se ha quitado la opción "Eliminar" ya que no tenía mucho sentido y así evitamos que se genere un registro de este módulo.
7818e5b

@ainaraRT
Copy link
Collaborator Author

ainaraRT commented Jan 9, 2025

Al crear un registro de un módulo, ya sea al crear o al duplicar, aparecen dos registros en el módulo de Monitorización de los cuales uno no es entendible

Mensaje ilegible tras duplicar

Corregido en 53a9798

@ainaraRT
Copy link
Collaborator Author

He leído un comentario de Alberto donde he interpretado que queremos que el módulo aparezca en Estudio, sin embargo, este no aparece.

Comentado con Alberto. "Al ser un módulo de uso exclusivo para Administradores y sin edición, no lo añadiremos en Estudio"

@ainaraRT
Copy link
Collaborator Author

Por último, el número de registros de este módulo va a crecer de forma muy rápida pudiendo dar problemas de rendimiento como en los registros auditados de flujos de trabajo

¿Se ha valorado crear alguna automatización para que se eliminen los registros de monitorización a nivel lógico pasado un tiempo?

Ya existe un programador que elimina la tabla cada primero de mes anteriores a los últimos 30 días:
image

@ManuSinergiaCRM
Copy link
Collaborator

Por último, el número de registros de este módulo va a crecer de forma muy rápida pudiendo dar problemas de rendimiento como en los registros auditados de flujos de trabajo
¿Se ha valorado crear alguna automatización para que se eliminen los registros de monitorización a nivel lógico pasado un tiempo?

Ya existe un programador que elimina la tabla cada primero de mes anteriores a los últimos 30 días: image

Tras comentarlo con Ainara, vemos que ya existe el planificador (Prune Tracker Tables) y que este depende de una variable de configuración (tracker_prune_interval) que si no está definida en config.php, tomará el valor por defecto de 30 días.

Propongo crear la variable en config.php para que sea más fácil identificar que existe dicha configuración y subir el tiempo a 90 días para aplicar el mismo criterio que hemos utilizado otras veces, que es el de almacenar la información de los 3 meses anteriores.

@ainaraRT
Copy link
Collaborator Author

Por último, el número de registros de este módulo va a crecer de forma muy rápida pudiendo dar problemas de rendimiento como en los registros auditados de flujos de trabajo
¿Se ha valorado crear alguna automatización para que se eliminen los registros de monitorización a nivel lógico pasado un tiempo?

Ya existe un programador que elimina la tabla cada primero de mes anteriores a los últimos 30 días: image

Tras comentarlo con Ainara, vemos que ya existe el planificador (Prune Tracker Tables) y que este depende de una variable de configuración (tracker_prune_interval) que si no está definida en config.php, tomará el valor por defecto de 30 días.

Propongo crear la variable en config.php para que sea más fácil identificar que existe dicha configuración y subir el tiempo a 90 días para aplicar el mismo criterio que hemos utilizado otras veces, que es el de almacenar la información de los 3 meses anteriores.

Añadido ee7936e

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.

Nueva funcionalidad - General - Módulo de registro de acceso y uso de SinergiaCRM
4 participants