PERINATAL - Revisar filtros y scroll listado perinatal #2857

maring0019 · 2023-04-28T16:09:42Z

Requerimiento

https://proyectos.andes.gob.ar/browse/PER-82

Funcionalidad desarrollada

Se cambia el contenido del campo "estado" por una condición porque sino solo descargaba en el csv los ausentes. De este modo el csv y el listado que se muestra en la aplicación coinciden.
Se saca el valor ' ' del campo paciente ya que no afecta en la descarga del csv.
Se cambia la hora de la fechaHastaEntrada para que incluya los documentos del día seleccionado. Para ello se usa otra variable fechaEntrada2 porque sino había problemas al cambiar fecha en el campo fechaHasta.
El problema del doble scroll que se pide en la tarea, se puede deber al zoom que se hace en el navegador. No se pudo reproducir ese error.

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

silviroa · 2023-04-28T17:24:30Z

USUARIO: mbenicio
BUILD NUMBER: 8801
CYPRESS RUN: 7971
TEST START: 2023-04-28T17:23:08.023Z
TOTAL: 358
SUCCESS: 348
FAIL: 8
SKIPPED: 2

silviroa · 2023-04-28T18:29:33Z

USUARIO: mbenicio
BUILD NUMBER: 8802
CYPRESS RUN: 7972
TEST START: 2023-04-28T18:29:20.896Z
TOTAL: 406
SUCCESS: 404
FAIL: 0
SKIPPED: 2

MarianoCampetella · 2023-05-12T12:18:40Z

Si bien los filtros funcionan bien, cuando hago la descarga del csv tanto en el módulo de perinatal como en bi-queries y agrego la fecha desde: 01/02/23 y la fecha hasta: 14/02/23 me descarga la siguiente cantidad de datos, los cuales no coinciden con los que se encuentran en la BD. Revisar el comportamiento de la query.

Se han realizado cambios.

MCele · 2023-05-15T16:19:20Z

src/app/modules/perinatal/services/carnet-perinatal.service.ts

@@ -63,7 +63,7 @@ export class CarnetPerinatalService extends ResourceBaseHttp {
 if (profesional) {
 params.profesional = (profesional as IProfesional).id;
 }
- params.fechaControl = this.queryDateParams(fechaDesde as Date, fechaHasta as Date);
+ params.fecha = this.queryDateParams(fechaDesde as Date, fechaHasta as Date);


Cuidado con esto:
la fecha es la fecha inicial en la que se iniciaron los controles de embarazo y "fechaControl" es la fecha de un control.
El objetivo del listado es poder filtrar por fechas de controles, no necesariamente tiene que ser la fecha de la columna "Fecha inicio" del listado. La tarea no pide que se modifique el objetivo de la query, si no que coincidan los controles listados con la descarga csv.
Mismo comentario al intentar modificar: "controles.fechaControl" por “fecha”.

Se han realizado cambios.

JuanIRamirez

Sugiero aumentar el tiempo de rebote [debounce] en "fechaHasta" ej: "2000", porque cambia rapidamente la fecha ingresada por tener la propiedad [min]="fechaDesdeEntrada".

El problema quedó resuelto. El motivo de ese problema no era el debounce sino la actualización de la fechaHastaEntrada en this.fechaHastaEntrada = moment(this.fechaHastaEntrada).endOf('day');

Fabio-Ramirez · 2023-09-05T14:53:19Z

src/app/modules/perinatal/components/listado-perinatal.component.ts

- public fechaHastaEntrada;
+ public fechaDesdeEntrada: any;
+ public fechaHastaEntrada: any;
+ public fechaHastaEntrada2: any;


Seria optimo para las buenas practicas de programacion en TS, evitaria utilizar en any y le asignaria un tipo de dato a estas variables, por ejemplo, como en este caso se trata de fecha, utilizaria Date como tipo. En el componente: fuera-agenda.component.ts hay un ejemplo.

Seria optimo para las buenas practicas de programacion en TS, evitaria utilizar en any y le asignaria un tipo de dato a estas variables, por ejemplo, como en este caso se trata de fecha, utilizaria Date como tipo. En el componente: fuera-agenda.component.ts hay un ejemplo.

Cambios hechos.

Fabio-Ramirez

Mari estaria correcta las correciones de las observaciones que te realice. Ahora me aparece el siguiente error.
https://www.loom.com/share/5ae83f9dcc724d84b1f9cb7f1335ca23

maring0019 · 2023-10-03T17:33:56Z

Mari estaria correcta las correciones de las observaciones que te realice. Ahora me aparece el siguiente error. https://www.loom.com/share/5ae83f9dcc724d84b1f9cb7f1335ca23

Fijarse que la base de datos esté activa y el microservice también.

MarianoCampetella · 2024-01-10T15:10:49Z

src/app/modules/perinatal/components/listado-perinatal.component.ts

@@ -123,14 +124,16 @@ export class ListadoPerinatalComponent implements OnInit {
 this.carnetPerinatalService.lastResults.next(null);
 this.carnetPerinatalService.paciente.next(this.paciente);
 this.carnetPerinatalService.fechaDesde.next(this.fechaDesdeEntrada);
- this.carnetPerinatalService.fechaHasta.next(this.fechaHastaEntrada);
+ if (this.fechaHastaEntrada) {
+ this.fechaHastaEntrada2 = moment(this.fechaHastaEntrada).endOf('day');


Corregir esta linea ya que fechaHastaEntrada2 se encuentra declarada como Date y se le esta asignando un moment()

maring0019 requested review from a team as code owners April 28, 2023 16:09

maring0019 requested review from ma7payne and AgosLizzi April 28, 2023 16:09

maring0019 force-pushed the PER-82 branch from 55ed07c to c7a8b11 Compare April 28, 2023 16:13

silviroa added the test fail label Apr 28, 2023

silviroa added test ok Los test estan ok and removed test fail labels Apr 28, 2023

maring0019 force-pushed the PER-82 branch 2 times, most recently from 2861d09 to 9d60d15 Compare May 8, 2023 17:26

MarianoCampetella added the changes requested Se solicitaron cambios label May 12, 2023

negro89 added the prioritario Tarea con prioridad alta label May 15, 2023

MCele requested changes May 15, 2023

View reviewed changes

maring0019 force-pushed the PER-82 branch from 9d60d15 to b468500 Compare May 23, 2023 15:20

maring0019 force-pushed the PER-82 branch from b468500 to 491fde1 Compare June 7, 2023 18:30

maring0019 force-pushed the PER-82 branch 10 times, most recently from f5f669e to 50e5fa9 Compare June 22, 2023 18:41

maring0019 force-pushed the PER-82 branch from 50e5fa9 to 41d67ba Compare June 30, 2023 14:37

JuanIRamirez self-requested a review August 16, 2023 14:56

JuanIRamirez requested changes Aug 16, 2023

View reviewed changes

maring0019 force-pushed the PER-82 branch 2 times, most recently from b6d8c4a to 154cf31 Compare August 17, 2023 16:16

maring0019 requested a review from JuanIRamirez August 17, 2023 16:29

github-actions bot added the has_conflicts label Aug 17, 2023

silviroa force-pushed the PER-82 branch from 154cf31 to ec65b64 Compare August 18, 2023 12:57

silviroa removed the has_conflicts label Aug 18, 2023

maring0019 force-pushed the PER-82 branch from ec65b64 to 0f4d167 Compare September 4, 2023 18:28

Fabio-Ramirez self-assigned this Sep 5, 2023

Fabio-Ramirez requested changes Sep 5, 2023

View reviewed changes

Fabio-Ramirez added changes requested Se solicitaron cambios and removed changes done labels Sep 5, 2023

maring0019 force-pushed the PER-82 branch from 0f4d167 to 4e2ac4a Compare September 6, 2023 17:29

maring0019 added changes done and removed changes requested Se solicitaron cambios labels Sep 6, 2023

maring0019 requested a review from Fabio-Ramirez September 6, 2023 17:39

Fabio-Ramirez reviewed Oct 3, 2023

View reviewed changes

Fabio-Ramirez added changes requested Se solicitaron cambios and removed changes done labels Oct 3, 2023

maring0019 requested a review from Fabio-Ramirez October 3, 2023 17:34

maring0019 added changes done and removed changes requested Se solicitaron cambios labels Oct 4, 2023

MarianoCampetella requested changes Jan 10, 2024

View reviewed changes

MarianoCampetella added changes requested Se solicitaron cambios and removed changes done labels Jan 10, 2024

MCele force-pushed the PER-82 branch from 4e2ac4a to e58aa75 Compare January 26, 2024 15:23

fix(PER): se corrigen parámetros

e58aa75

github-actions bot added the has_conflicts label Jan 31, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

PERINATAL - Revisar filtros y scroll listado perinatal #2857

PERINATAL - Revisar filtros y scroll listado perinatal #2857

maring0019 commented Apr 28, 2023 •

edited

silviroa commented Apr 28, 2023

silviroa commented Apr 28, 2023

MarianoCampetella commented May 12, 2023 •

edited by maring0019

MCele May 15, 2023 •

edited

maring0019 Jun 30, 2023

JuanIRamirez left a comment •

edited by maring0019

Fabio-Ramirez Sep 5, 2023

maring0019 Sep 6, 2023

Fabio-Ramirez left a comment •

edited

maring0019 commented Oct 3, 2023

MarianoCampetella Jan 10, 2024

PERINATAL - Revisar filtros y scroll listado perinatal #2857

Are you sure you want to change the base?

PERINATAL - Revisar filtros y scroll listado perinatal #2857

Conversation

maring0019 commented Apr 28, 2023 • edited

Requerimiento

Funcionalidad desarrollada

UserStory llegó a completarse

Requiere actualizaciones en la base de datos

Requiere actualizaciones en la API

Requiere actualizaciones en andes-test-integracion

silviroa commented Apr 28, 2023

silviroa commented Apr 28, 2023

MarianoCampetella commented May 12, 2023 • edited by maring0019

MCele May 15, 2023 • edited

Choose a reason for hiding this comment

maring0019 Jun 30, 2023

Choose a reason for hiding this comment

JuanIRamirez left a comment • edited by maring0019

Choose a reason for hiding this comment

Fabio-Ramirez Sep 5, 2023

Choose a reason for hiding this comment

maring0019 Sep 6, 2023

Choose a reason for hiding this comment

Fabio-Ramirez left a comment • edited

Choose a reason for hiding this comment

maring0019 commented Oct 3, 2023

MarianoCampetella Jan 10, 2024

Choose a reason for hiding this comment

maring0019 commented Apr 28, 2023 •

edited

MarianoCampetella commented May 12, 2023 •

edited by maring0019

MCele May 15, 2023 •

edited

JuanIRamirez left a comment •

edited by maring0019

Fabio-Ramirez left a comment •

edited