IN - Reporte pedido de medicamentos #2997

aldoEMatamala · 2024-04-08T12:25:05Z

Requerimiento

https://proyectos.andes.gob.ar/browse/IN-574

Funcionalidad desarrollada

Reporte de los pedidos de medicamentos de un sector para realizar el control de entrega de medicación
Extraccion de una planilla con la siguiente información: DNI paciente / sector / unidad org./ cama actual/ los datos del medicamento, filtros por sector, unidad organizativa y por nombre de medicamento para un efector.
Descarga en csv

UserStory llegó a completarse

Si
No
No corresponde

Requiere actualizaciones en la base de datos

Si
No

Requiere actualizaciones en la API

Requiere actualizaciones en andes-test-integracion

Si
No

negro89

chicos, dos cositas por ahora, ademas de lo charlado en la demo:

Se está pudiendo acceder por url por fuera de la capa farmaceutica
El mensaje del listado vacío si no se está filtrando no debería decir que no hay resultados, sino sugerir como empezar a interactuar

El resto de 10 👍🏼

aldoEMatamala · 2024-04-16T13:57:08Z

2. está filtrando no debería decir que no hay resultados, sino sugerir como empezar a interactuar

chicos, dos cositas por ahora, ademas de lo charlado en la demo:
1. Se está pudiendo acceder por url por fuera de la capa farmaceutica

2. El mensaje del listado vacío si no se está filtrando no debería decir que no hay resultados, sino sugerir como empezar a interactuar
El resto de 10 👍🏼
gracias por la revisión!
1- Queda la redireccion cuando no se encuentra en capa distinta a 'farmaceutica' y se quiere acceder al listado.
2- Se modificó la leyenda de listado vacío a

Saludos!

MarianoCampetella · 2024-04-19T12:21:08Z

En la primera foto se observa que el filtro de paciente no esta funcionando como corresponde ya que al ingresar un apellido que no se encuentra en el listado sigue devolviendo todos los medicamentos registrados en el día.
En la segunda foto me queda la duda si no será mejor cambiar la leyenda cuando no obtenemos ningún resultado a partir de la búsqueda de cualquiera de los filtros. En caso de que se pueda implementar dos leyendas serian de la siguiente forma:

Si no existen medicamentos registrados en el día de la fecha podría decir la leyenda que se encuentra ahora.
Si no existen medicamentos en base a los filtros utilizados entonces mostrar una leyenda que diga: No hay resultados para esta búsqueda - Edite algún filtro para realizar una nueva.

aldoEMatamala · 2024-05-21T19:07:30Z

En la primera foto se observa que el filtro de paciente no esta funcionando como corresponde ya que al ingresar un apellido que no se encuentra en el listado sigue devolviendo todos los medicamentos registrados en el día. En la segunda foto me queda la duda si no será mejor cambiar la leyenda cuando no obtenemos ningún resultado a partir de la búsqueda de cualquiera de los filtros. En caso de que se pueda implementar dos leyendas serian de la siguiente forma:
* Si no existen medicamentos registrados en el día de la fecha podría decir la leyenda que se encuentra ahora.

* Si no existen medicamentos en base a los filtros utilizados entonces mostrar una leyenda que diga: No hay resultados para esta búsqueda - Edite algún filtro para realizar una nueva.

Gracias por la corrección!

Queda corregido cuando se modificaba el filtro de paciente y filtraba mal.
Modificada la leyenda a "No hay resultados para esta búsqueda"

aldoEMatamala · 2024-05-28T16:08:53Z

negro89 · 2024-06-11T15:09:07Z

Aldinho, fijate que debes haber pisado los cambios de este commit 9658734 porque en la rama actual no se ven reflejados.
Agrego como observación que en la pantalla del listado de medicamentos, si se actualiza el navegador vuelve al mapa de camas pero en la capa medica.
El resto ok 👌🏼

negro89 · 2024-06-13T12:56:14Z

...pps/rup/mapa-camas/views/listado-internacion-capas/listado-medicamentos-capas.component.html

+ </plex-wrapper>
+ <div *ngIf="!(listaMedicamentos$ | async)?.length" class="mt-5">
+ <plex-label class="flex-column" icon="magnify" type="info" justify="center" size="xl" direction="column"
+ titulo="No hay resultados para esta búsqueda"


queda correjido lo de al refrescar la vista que se iba a otra pantalla, también se agrego la necesidad de tener el permiso de capa farmaceutica para acceder al listado.

Excelente 👍🏼 con respecto al cartel que se muestra inicialmente en el listado, debés tener algo cacheado porque en el codigo sigue figurando "No hay resultados para esta búsqueda"

Aldinho, fijate que debes haber pisado los cambios de este commit 9658734 porque en la rama actual no se ven reflejados. Agrego como observación que en la pantalla del listado de medicamentos, si se actualiza el navegador vuelve al mapa de camas pero en la capa medica. El resto ok 👌🏼

Buenas ! gracias por la revisión.
-Se modifican leyendas cuando arroja listado vacio en función de la existencia de pedidos de medicamentos en el día de la fecha:
=> si hay pedidos pero no ninguno conhincide con los filtros aplicados

=> si no hay ningún pedido en el día.

aldoEMatamala · 2024-06-13T18:43:30Z

aldoEMatamala · 2024-06-14T18:13:22Z

https://cloud.cypress.io/projects/xr7gft/runs/10165/overview?roarHideRunsWithDiffGroupsAndTags=1

MarianoCampetella · 2024-04-18T18:57:45Z

src/app/apps/rup/mapa-camas/services/mapa-camas.http.ts

+ if (organizacionID) {
+ params = {
+ ambito,
+ desde,
+ hasta,
+ organizacionID
+ };
+ } else {
+ params = {
+ ambito,
+ desde,
+ hasta
+ };


Para no repetir codigo podemos crear una nueva variable let params = { ambito, desde, hasta } y dentro de una condición podemos agregar el otro atributo faltante en caso de que exista el id de la organización.

quedó como lo conversado. Gracias!

MarianoCampetella · 2024-07-12T13:27:32Z

src/app/apps/rup/mapa-camas/services/plan-indicaciones.service.ts

+import { BehaviorSubject, EMPTY, Observable, combineLatest, of, Subscription, forkJoin } from 'rxjs';
+import { auditTime, filter, map, mergeMap, switchMap } from 'rxjs/operators';


Las variables Subscription al igual que filter no estan siendo utilizadas, lo cual deben ser eliminadas.

MarianoCampetella · 2024-07-12T13:36:15Z

...pps/rup/mapa-camas/views/listado-internacion-capas/listado-medicamentos-capas.component.html

+ titulo="No se ha registrado solicitudes de medicamentos en el día de la fecha"
+ subtitulo="Se mostraran los medicamentos cargados desde el plan de indicaciones de los pacientes">


En titulo dejaría: No se han registrado....
En subtitulo falta el acento a mostrarán.

MarianoCampetella · 2024-07-12T14:02:48Z

.../apps/rup/mapa-camas/views/listado-internacion-capas/listado-medicamentos-capas.component.ts

+ public fechaDesdeEntrada;
+ public fechaHastaEntrada;
+ public listadoActual: any[];
+ public hayHoy = true;


Múltiples variables como fechaDesdeEntrada, fechaHastaEntrada, hayHoy,selectedPrestacion$ y fechaControl no estan siendo utilizadas, al igual que Iprestaciones de la linea 7.

MarianoCampetella · 2024-07-12T14:04:51Z

También falta corregir uno de los comentarios de arriba, el de la variable params.

negro89 · 2024-07-17T13:41:29Z

...pps/rup/mapa-camas/views/listado-internacion-capas/listado-medicamentos-capas.component.html

+ internacion.estadoActual?.tipo}}
+ </span>
+ <span *ngIf="internacion.estadoActual?.tipo==='bypass'">
+ {{"bypass("+(internacion.estadoActual?.verificacion?.estado==null?


Por mi está joya. El unico detalle que veo es la mayuscula en bypass (que si está en el estado "Aceptada" y el espacio antes del paréntesis) que lo puedo sumar yo al momento del deploy.

# [5.144.0](v5.143.0...v5.144.0) (2024-07-24) ### Bug Fixes * **RUP:** repara acceso a vistas de registros HUDS ([#3037](#3037)) ([832a9fb](832a9fb)) ### Features * **CIT-272:** Ventanilla-Solicitudes: visualizar estado asignada ([#3026](#3026)) ([f05ca69](f05ca69)) * **CIT-273:** Visualizar edad del paciente ([#3025](#3025)) ([802e18b](802e18b)) * **IN-574:** Reporte indicaciones de medicamentos ([#2997](#2997)) ([faf8d14](faf8d14)) * **IN-599:** Cambiar orden en Registros médicos ([#3029](#3029)) ([1f35c47](1f35c47)) * **RUP:** implementa habilitacion de prestaciones a profesional ([#3014](#3014)) ([ba954c2](ba954c2))

aldoEMatamala requested review from a team as code owners April 8, 2024 12:25

aldoEMatamala requested review from MCele and AgosLizzi April 8, 2024 12:25

aldoEMatamala force-pushed the IN-574 branch 4 times, most recently from de50587 to 718e7d5 Compare April 9, 2024 18:49

negro89 reviewed Apr 11, 2024

View reviewed changes

aldoEMatamala force-pushed the IN-574 branch from 718e7d5 to 9658734 Compare April 16, 2024 13:53

MarianoCampetella added the changes requested Se solicitaron cambios label Apr 19, 2024

aldoEMatamala force-pushed the IN-574 branch 3 times, most recently from 1315802 to 3b93373 Compare May 21, 2024 19:03

aldoEMatamala added changes done and removed changes requested Se solicitaron cambios labels May 21, 2024

aldoEMatamala requested a review from negro89 May 21, 2024 19:08

aldoEMatamala added the test ok Los test estan ok label May 28, 2024

aldoEMatamala force-pushed the IN-574 branch 3 times, most recently from a785a07 to 8a59f6a Compare May 28, 2024 16:17

negro89 added changes requested Se solicitaron cambios and removed changes done labels Jun 11, 2024

negro89 reviewed Jun 13, 2024

View reviewed changes

aldoEMatamala requested a review from negro89 June 14, 2024 15:05

aldoEMatamala force-pushed the IN-574 branch from f3b9b75 to 19d0317 Compare June 14, 2024 15:18

negro89 approved these changes Jun 19, 2024

View reviewed changes

github-actions bot added the has_conflicts label Jun 26, 2024

aldoEMatamala force-pushed the IN-574 branch from 19d0317 to 847f0a1 Compare June 28, 2024 15:16

aldoEMatamala removed the has_conflicts label Jun 28, 2024

MarianoCampetella requested changes Jul 12, 2024

View reviewed changes

MarianoCampetella added changes requested Se solicitaron cambios and removed changes done labels Jul 12, 2024

aldoEMatamala force-pushed the IN-574 branch 3 times, most recently from 45b142a to 55caf6f Compare July 16, 2024 14:10

aldoEMatamala requested review from MarianoCampetella and negro89 July 16, 2024 14:19

aldoEMatamala added changes done and removed changes requested Se solicitaron cambios labels Jul 16, 2024

negro89 approved these changes Jul 17, 2024

View reviewed changes

MarianoCampetella approved these changes Jul 18, 2024

View reviewed changes

negro89 force-pushed the IN-574 branch from 55caf6f to 1f019d3 Compare July 18, 2024 15:59

feat(IN-574): Reporte indicaciones de medicamentos

761af22

negro89 force-pushed the IN-574 branch from 1f019d3 to 761af22 Compare July 18, 2024 16:03

negro89 added Aprobado and removed changes done labels Jul 18, 2024

negro89 merged commit faf8d14 into master Jul 18, 2024
2 checks passed

negro89 deleted the IN-574 branch July 18, 2024 16:07

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

IN - Reporte pedido de medicamentos #2997

IN - Reporte pedido de medicamentos #2997

aldoEMatamala commented Apr 8, 2024 •

edited

Loading

negro89 left a comment

aldoEMatamala commented Apr 16, 2024

MarianoCampetella commented Apr 19, 2024

aldoEMatamala commented May 21, 2024

aldoEMatamala commented May 28, 2024

negro89 commented Jun 11, 2024

negro89 Jun 13, 2024

aldoEMatamala Jun 14, 2024 •

edited

Loading

aldoEMatamala commented Jun 13, 2024

aldoEMatamala commented Jun 14, 2024

MarianoCampetella Apr 18, 2024

aldoEMatamala Jul 16, 2024

MarianoCampetella Jul 12, 2024

aldoEMatamala Jul 16, 2024

MarianoCampetella Jul 12, 2024

aldoEMatamala Jul 16, 2024

MarianoCampetella Jul 12, 2024

aldoEMatamala Jul 16, 2024

MarianoCampetella commented Jul 12, 2024

negro89 Jul 17, 2024

		import { BehaviorSubject, EMPTY, Observable, combineLatest, of, Subscription, forkJoin } from 'rxjs';
		import { auditTime, filter, map, mergeMap, switchMap } from 'rxjs/operators';

		titulo="No se ha registrado solicitudes de medicamentos en el día de la fecha"
		subtitulo="Se mostraran los medicamentos cargados desde el plan de indicaciones de los pacientes">

IN - Reporte pedido de medicamentos #2997

IN - Reporte pedido de medicamentos #2997

Conversation

aldoEMatamala commented Apr 8, 2024 • edited Loading

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

negro89 left a comment

Choose a reason for hiding this comment

aldoEMatamala commented Apr 16, 2024

MarianoCampetella commented Apr 19, 2024

aldoEMatamala commented May 21, 2024

aldoEMatamala commented May 28, 2024

negro89 commented Jun 11, 2024

Choose a reason for hiding this comment

aldoEMatamala Jun 14, 2024 • edited Loading

Choose a reason for hiding this comment

aldoEMatamala commented Jun 13, 2024

aldoEMatamala commented Jun 14, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MarianoCampetella commented Jul 12, 2024

Choose a reason for hiding this comment

aldoEMatamala commented Apr 8, 2024 •

edited

Loading

aldoEMatamala Jun 14, 2024 •

edited

Loading