SinergiaTIC / SinergiaDA

SinergiaDA is an open source analytics tool that fully integrates with SinergiaCRM
GNU Affero General Public License v3.0
4 stars 0 forks source link

Bug - API - Fallo en updateModel al actualizar pertenencia a grupos de seguridad #135 #143

Closed jortilles closed 4 months ago

jortilles commented 6 months ago

Descripción del Cambio

Refactorización para anadir los grupos.

Issue(s) resuelto(s)

Pruebas a realizar para validar el cambio

Mover los grupos de seguridad de un usuario y comprobar el funcionamiento. Tal y como se describe en el issue.

asafJortilles commented 5 months ago

@PaulaaSTIC Al hacer el cambio en la forma en la que tomamos los datos de grupos desde el CRM, @juanSTIC se dio cuenta que el módulo de crear un grupo SDA_BASIC era innecesario. Es posible que Juan lo haya eliminado en vuestro local y por eso peta en esas 2 lineas. Puedes flaguear ambas lineas donde se hace la llamada a este modulo para probar?

asafJortilles commented 5 months ago

Error entre usuarios. En el CRM hay un usuario u1 cuyo correo es u1@test.com, y en SDA hay un usuario u1 con el correo u1@u1.com. Los permisos que vienen del CRM se están actualizando en el usuario u1@u1.com en lugar de en el usuario del CRM.

Debería comprobarse tanto el nombre como el correo electrónico a la hora de establecer permisos en un usuario. Si elimino el usuario u1@test.com deel CRM en SDA me lo genera de nuevo con los permisos correctos.

image

image image

@PaulaaSTIC no he sido capaz de reproducir el error, he intentado crear el usuario con distinto mail pero mismo user y he conseguido mantener inalterado el usuario en SDA y traer el usuario del CRM con sus permisos pertinentes.

PaulaaSTIC commented 5 months ago

Error al ejecutar el UpdateModel image

jortilles commented 4 months ago

Esto ya se puede volver a validar. Ya esta resuelto.

jortilles commented 4 months ago

@PaulaaSTIC @juanSTIC puedes volver a validar, por favor.

jortilles commented 4 months ago

Ya se ha sincronizado

juanSTIC commented 4 months ago

Sincronizada con la última versión de reporting