Skip to content

Conversation

@ma7payne
Copy link
Contributor

@ma7payne ma7payne commented Mar 21, 2025

Requerimiento

https://proyectos.andes.gob.ar/browse/RUP-452
https://proyectos.andes.gob.ar/browse/HUDS-156

Funcionalidad desarrollada

  1. Se implementa un nuevo gestor de renovacion y suspensión de recetas
  2. Se evita el agregado de medicamentos duplicados en prestación de RUP
  3. Se actualizan datos del profesional y efector cuando se realiza la renovación de una receta.

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

@ma7payne ma7payne requested review from a team as code owners March 21, 2025 15:43
@ma7payne ma7payne requested review from AgosLizzi and MCele March 21, 2025 15:43
@ma7payne ma7payne force-pushed the RUP-452 branch 6 times, most recently from d8cb327 to 9d0fa0c Compare April 3, 2025 11:26
Copy link
Contributor

@AgosLizzi AgosLizzi left a comment

Choose a reason for hiding this comment

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

Mati, El botón volver lo pasaría a type="danger" y le agregaría tooltip, por las dudas.

@Fabio-Ramirez Fabio-Ramirez added the changes requested Se solicitaron cambios label Apr 4, 2025
Copy link
Contributor

@MCele MCele left a comment

Choose a reason for hiding this comment

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

Mati en RUP veo un pequeño scroll en cada concepto snomed que se lista en el buscador
image
Otro detalle que veo es que a veces no se visualiza el help de confirmación de medicamentos a renovar
image
Pasos para esto último: Selecciono las recetas a renovar y botón verde de "Renovar", aparece el plex-help, cierro help. modifico medicamentos, nuevamente boton "Renovar" y no se visualiza el help.

@ma7payne ma7payne force-pushed the RUP-452 branch 3 times, most recently from f6dd028 to 46d4c43 Compare April 15, 2025 15:58
@ma7payne
Copy link
Contributor Author

ma7payne commented Apr 15, 2025

Listos los cambios pedidos por @AgosLizzi y @MCele, a excepción de lo mencionado en el plex-help que no lo pude replicar, pero tampoco encuentro algún problema.

@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels Apr 15, 2025
@MCele
Copy link
Contributor

MCele commented Apr 21, 2025

Listos los cambios pedidos por @AgosLizzi y @MCele, a excepción de lo mencionado en el plex-help que no lo pude replicar, pero tampoco encuentro algún problema.

okis Mati, si no alcanza con los pasos que te dejé al final del comentario, avisame y lo vemos juntos

@MCele MCele requested a review from AgosLizzi April 21, 2025 20:01
@JuanIRamirez JuanIRamirez self-requested a review April 29, 2025 18:23
@JuanIRamirez
Copy link
Contributor

Al renovar generar una nueva Receta (POST)
andes/api#2047

@ma7payne
Copy link
Contributor Author

Compa, a lo que charlamos recien te dejo el videito de las cositas que noto...

recording-29.mp4
Por consola, el error de un "style", y por otro lado el error de api (esto quizas venga por otro lado).

Gracias @Fabio-Ramirez . Vemos que el error viene por otro lado asi que queda a resolver por fuera.

@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels Jul 10, 2025
Copy link
Contributor

@agustin1996ra agustin1996ra left a comment

Choose a reason for hiding this comment

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

Funciona bien el circuito, lo unico que me encontre es que no se ve el icono de receta.
image

@ma7payne
Copy link
Contributor Author

Funciona bien el circuito, lo unico que me encontre es que no se ve el icono de receta. image

Genial @agustin1996ra ! La falta del ícono es probablemente la versión desactualizada de plex (>8.0.13) o andes icons (1.1.28).

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants