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.

Sign up for GitHub

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

Jump to bottom

IN - Reporte pedido de medicamentos #2997

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

IN - Reporte pedido de medicamentos #2997

wants to merge 1 commit into from

Conversation

aldoEMatamala
Copy link
Contributor

@aldoEMatamala aldoEMatamala commented Apr 8, 2024 •
edited

Requerimiento

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

Funcionalidad desarrollada

  1. Reporte de los pedidos de medicamentos de un sector para realizar el control de entrega de medicación
  2. 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.
  3. 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

@aldoEMatamala aldoEMatamala requested review from a team as code owners April 8, 2024 12:25
negro89
negro89 reviewed Apr 11, 2024
View reviewed changes
Copy link
Contributor

@negro89 negro89 left a comment

Choose a reason for hiding this comment

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

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 👍🏼

@aldoEMatamala
Copy link
Contributor Author

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
imagen
Saludos!

@MarianoCampetella
Copy link
Contributor

image
image

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.

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label Apr 19, 2024
@aldoEMatamala aldoEMatamala force-pushed the IN-574 branch 3 times, most recently from 1315802 to 3b93373 Compare May 21, 2024 19:03
@aldoEMatamala
Copy link
Contributor Author

image image

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 aldoEMatamala added changes done and removed changes requested Se solicitaron cambios labels May 21, 2024
@aldoEMatamala
Copy link
Contributor Author

imagen

@aldoEMatamala aldoEMatamala added the test ok Los test estan ok label May 28, 2024
@aldoEMatamala aldoEMatamala force-pushed the IN-574 branch 3 times, most recently from a785a07 to 8a59f6a Compare May 28, 2024 16:17
@negro89
Copy link
Contributor

negro89 commented Jun 11, 2024

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 negro89 added changes requested Se solicitaron cambios and removed changes done labels Jun 11, 2024
@negro89 negro89 added the changes requested Se solicitaron cambios label Jun 11, 2024
@aldoEMatamala
Copy link
Contributor Author

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 👌🏼

gracias por la corrección!

  • 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.

@aldoEMatamala aldoEMatamala added changes done and removed changes requested Se solicitaron cambios labels Jun 13, 2024
negro89
negro89 reviewed Jun 13, 2024
View reviewed changes
...pps/rup/mapa-camas/views/listado-internacion-capas/listado-medicamentos-capas.component.html Outdated
</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"
Copy link
Contributor

Choose a reason for hiding this comment

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

  • 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"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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
imagen
=> si no hay ningún pedido en el día.
Captura de pantalla de 2024-06-28 11-55-18

@aldoEMatamala
Copy link
Contributor Author

imagen

@aldoEMatamala
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@negro89 negro89 negro89 approved these changes

@MCele MCele Awaiting requested review from MCele MCele is a code owner automatically assigned from andes/code-review

@AgosLizzi AgosLizzi Awaiting requested review from AgosLizzi AgosLizzi is a code owner automatically assigned from andes/code-review

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
changes done test ok Los test estan ok
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@aldoEMatamala @MarianoCampetella @negro89