Skip to content

IN - Filtro por unidad organizativa#3330

Open
nicolasarana wants to merge 2 commits intomasterfrom
IN-682
Open

IN - Filtro por unidad organizativa#3330
nicolasarana wants to merge 2 commits intomasterfrom
IN-682

Conversation

@nicolasarana
Copy link
Copy Markdown
Contributor

@nicolasarana nicolasarana commented Apr 8, 2026

Requerimiento

IN-682

Funcionalidad desarrollada

  1. El problema era que el filtro solo buscaba en la unidad de ingreso (idPrestacion), pero la tabla (y el ordenamiento) también consideran la unidad del movimiento de cama actual (estadosCama). Esto causaba que, si un paciente era trasladado, el filtro dejara de encontrarlo por la unidad que se veía en la tabla.

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

  • [] Si
  • No

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

@nicolasarana nicolasarana requested review from a team as code owners April 8, 2026 13:56
@Fabio-Ramirez
Copy link
Copy Markdown
Contributor

Compa, en base a lo que analizamos recientemente, te dejo imagen en la que se continúa visualizando una unidad organizativa diferente a la correspondiente al último movimiento registrado.

image

@Fabio-Ramirez Fabio-Ramirez added the changes requested Se solicitaron cambios label Apr 16, 2026
@nicolasarana nicolasarana requested a review from a team as a code owner April 17, 2026 12:56
@nicolasarana
Copy link
Copy Markdown
Contributor Author

Compa, en base a lo que analizamos recientemente, te dejo imagen en la que se continúa visualizando una unidad organizativa diferente a la correspondiente al último movimiento registrado.

image

ya fue corregido, gracias por la revisión

@nicolasarana nicolasarana added changes done and removed changes requested Se solicitaron cambios labels Apr 17, 2026
return ultimoEstadoConPaciente?.unidadOrganizativa?.term || internacion.idPrestacion?.unidadOrganizativa?.term;
}

getUltimaUnidadOrganizativaConceptId(internacion: any): string {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Al parecer esta función no esta siendo llamada en ningún lado. En caso de que se fuera a utilizar en algún futuro entonces se puede dejar si no eliminarla.

}
let ultimoEstadoConPaciente: any = null;
for (const cama of estadosCama) {
if (cama.estados && cama.estados.length > 0) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Esta condición la podemos simplificar asi: if (cama.estados?.length)

}
let ultimoEstadoConPaciente: any = null;
for (const cama of estadosCama) {
if (cama.estados && cama.estados.length > 0) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Para simplificar un poco la condición podemos dejarla como:
if (cama.estados?.length)

@MarianoCampetella MarianoCampetella added changes requested Se solicitaron cambios and removed changes done labels Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changes requested Se solicitaron cambios

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants