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

Incidencia - API+APP - Error al reconstruir fuente de datos SinergiaCRM #233

Open
PaulaaSTIC opened this issue Oct 17, 2024 · 4 comments · Fixed by #236
Open

Incidencia - API+APP - Error al reconstruir fuente de datos SinergiaCRM #233

PaulaaSTIC opened this issue Oct 17, 2024 · 4 comments · Fixed by #236
Assignees
Labels
bug Something isn't working Update Model

Comments

@PaulaaSTIC
Copy link
Collaborator

Descripción

Se ha detectado que el PR #215 introduce un error en el archivo cleanModel.ts, que provoca que si se ha eliminado la fuente de datos de SinergiaCRM de MongoDB, se produzca un fallo durante la reconstrucción. Como consecuencia, la fuente no se regenera y el proceso de updateModel genera un error.
Este es el fragmento introducido en el PR:

model_granted_roles.forEach( r=> {
      r.source = 'update_model';
  }
  );

  // Recuperando los permisos provenientes de SinergiaDA 
  // la propiedad source --> "EDA" indica que el permiso proviene de la applicacion y no de la base de datos
  const userRoles = mgsmap[0].filter( (r:any) => {
      return r?.source === 'SDA' && !r.groupsName.find( e => e.startsWith('SDA_'))
  });

  // Agregando los permisos agregados previamente en la aplicacion. 
  const all_roles =   [ ...model_granted_roles, ...userRoles];   
  main_model.ds.metadata.model_granted_roles = all_roles;

  return main_model;

Información adicional

Este error puede observarse en reporting, ya que el PR mencionado ya está incorporado a la rama.

@PaulaaSTIC PaulaaSTIC added bug Something isn't working Update Model labels Oct 17, 2024
@jortilles
Copy link
Collaborator

@ronaldchavezjortilles puedes echarle un vistazo?

@ronaldchavezjortilles
Copy link
Collaborator

Se realizo el cambio de SDA por SCRM, listo para verificar.

@ronaldchavezjortilles
Copy link
Collaborator

Se corrigió el issue, listo para validar.

@jortilles
Copy link
Collaborator

Se crea el pr #236

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Update Model
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants