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

(CIT) script: listaEspera corregir efector #1989

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

JuanIRamirez
Copy link
Contributor

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-350

Funcionalidad desarrollada

  1. Recorre listaEspera.demandas.organizacion.nombre vacío y le pone nombre de organización.
  2. Recorreo authUsers.organizaciones.nombre vacío y le pone nombre de organización.

UserStories llegó a completarse

  • Si
  • No

Requiere actualizaciones en la base de datos

  • Si
  • No

@JuanIRamirez JuanIRamirez requested review from a team as code owners November 6, 2024 16:13
@github-actions github-actions bot added the script Necesidad de aplicar un scripts label Nov 6, 2024

try {
const listaEsperaSinOrg: any[] = await listaEspera.find({ 'demandas.organizacion.nombre': '' });
for (const listaespera of listaEsperaSinOrg) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Juan el pr funciona correctamente. A modo de sugerencia, probé el caso (que se puede dar en alguna ocación) de que no exista directamente el dato "nombre", y este script no lo contempla. Por si te sirve te recomendaria agregar un control, donde realice la busqueda de aquellos registros que no tengan el dato "nombre".

}
await listaEspera.findByIdAndUpdate(listaespera.id, listaespera);
}
const authUsersSinOrg: any[] = await AuthUsers.find({ 'organizaciones.nombre': '' });
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Se podría tener en cuenta el comentario anterior.

@Fabio-Ramirez Fabio-Ramirez added the changes requested Se solicitaron cambios label Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested Se solicitaron cambios script Necesidad de aplicar un scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants