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

RUP - Receta : modificar molécula #3005

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

RUP - Receta : modificar molécula #3005

wants to merge 2 commits into from

Conversation

aldoEMatamala
Copy link
Contributor

@aldoEMatamala aldoEMatamala commented May 3, 2024 •
edited by JuanIRamirez
Loading

Requerimiento

https://proyectos.andes.gob.ar/browse/RUP-395
https://proyectos.andes.gob.ar/browse/RUP-400
https://proyectos.andes.gob.ar/browse/RUP-427
https://proyectos.andes.gob.ar/browse/MISC-380

Funcionalidad desarrollada

https://proyectos.andes.gob.ar/browse/RUP-395

  1. Se modifica la carga de medicamentos
  2. Se quita restricción de 2 medicamentos por receta
  3. Se quitan opciones de triplicado/duplicado/simple

https://proyectos.andes.gob.ar/browse/RUP-400
1- Se agregar la cantidad de unidades (campo numérico).
2- Se campo dosis diaria

https://proyectos.andes.gob.ar/browse/RUP-427
1- Cambio titulo de Indicaciones por "Indicaciones diarias".
2- En los campos frecuencia se agrega un combo de 1 a 24 por horas, y en duración que por día.
3- Se modifican nombres de los atributos para que coincidan con los que se guarda el registro.

https://proyectos.andes.gob.ar/browse/MISC-380
1- Se agrega campo 'Dosis' (medicamento.dosisDiaria.dosis)

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 May 3, 2024 14:12
@aldoEMatamala aldoEMatamala requested review from ma7payne and negro89 May 3, 2024 14:12
@aldoEMatamala
Copy link
Contributor Author

imagen

@aldoEMatamala aldoEMatamala added the test ok Los test estan ok label May 14, 2024
@MarianoCampetella
Copy link
Contributor

Habría que eliminar las lineas 30 hasta la 35 ya que no se utilizan en ningún lado de la clase: recetaMedica.component.ts al que Output, Input, EventEmitter dentro del import de la linea 1.

@MarianoCampetella
Copy link
Contributor

1- Me queda la duda si siempre se tiene que mostrar el badge de "simple"
2- Al momento de agregar una medicación se podría mostrar un toast de que la acción se realizó de forma correcta ya que puede ser confuso cuando se agrega una nueva medicación y no solamente quedan todos los campos obligatorios en rojo sino que dependiendo de las dimensiones de la pantalla es necesario tener que scrollear para ver el medicamento.
3- Lo mismo que el comentario dos pero para eliminar el medicamento, agregar un toast indicando que la acción de eliminar se hizo de forma correcta.
image

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label Jun 4, 2024
@aldoEMatamala
Copy link
Contributor Author

aldoEMatamala commented Jun 18, 2024 •
edited
Loading

1- Me queda la duda si siempre se tiene que mostrar el badge de "simple" 2- Al momento de agregar una medicación se podría mostrar un toast de que la acción se realizó de forma correcta ya que puede ser confuso cuando se agrega una nueva medicación y no solamente quedan todos los campos obligatorios en rojo sino que dependiendo de las dimensiones de la pantalla es necesario tener que scrollear para ver el medicamento. 3- Lo mismo que el comentario dos pero para eliminar el medicamento, agregar un toast indicando que la acción de eliminar se hizo de forma correcta. image

campe! gracias por la revision:

  • Se quita el badge de simple (consultado)
  • Se agregan toast al agregar y quitar medicamento
  • El PR queda en espera porque quedan por definir requerimientos (consultado)

@aldoEMatamala aldoEMatamala added En pausa Temporalmente pausado changes done and removed changes requested Se solicitaron cambios labels Jun 18, 2024
@aldoEMatamala aldoEMatamala force-pushed the RUP-395 branch 5 times, most recently from d3cabfe to 7f04d6f Compare September 24, 2024 18:43
@aldoEMatamala aldoEMatamala removed the En pausa Temporalmente pausado label Sep 26, 2024
@MCele
Copy link
Contributor

MCele commented Oct 1, 2024

Aldo veo que el resumen de los medicamentos muestra los badge raros
image

@aldoEMatamala
Copy link
Contributor Author

Aldo veo que el resumen de los medicamentos muestra los badge raros image

imagen
corregido! gracias por la revisión

@negro89 negro89 added the dependencies Requiere incorporar otra funcionalidad label Oct 18, 2024
@negro89
Copy link
Contributor

negro89 commented Oct 18, 2024 •
edited
Loading

Dependencia (a la espera de) la tarea Modificaciones molécula RECETA

@aldoEMatamala
Copy link
Contributor Author

image
para visualizar hint en lugar correcto se hizo uso de
"@andes/plex": "^8.0.5"

@aldoEMatamala aldoEMatamala removed the dependencies Requiere incorporar otra funcionalidad label Oct 21, 2024
@aldoEMatamala
Copy link
Contributor Author

Dependencia (a la espera de) la tarea Modificaciones molécula RECETA

se remueve la etiqueta dependecia por actualización de tarea

@JuanIRamirez JuanIRamirez self-requested a review October 22, 2024 15:35
@JuanIRamirez
Copy link
Contributor

Debería decir Cantidad y NO Cantidad de comprimidos, tbn en la Interface debería se cantidad y NO cantComprimidos.
Sigue teniendo la restricción de 2 medicamentos máximo.

@JuanIRamirez
Copy link
Contributor

JuanIRamirez commented Oct 22, 2024 •
edited
Loading

image

Fijate que dice: Frecuencia: 8Hs Comprimidos(s) por 3 días:
Debería decir algo así: Frecuencia: c/ 8 Hs. p/ 3 días.

@aldoEMatamala
Copy link
Contributor Author

aldoEMatamala commented Oct 22, 2024 •
edited
Loading

Fijate que dice: Frecuencia: 8Hs Comprimidos(s) por 3 días:
Debería decir algo así: Frecuencia: c/ 8 Hs. p/ 3 días.

Corregidas las leyendas al agregar medicamento y al validar consulta
image

@aldoEMatamala
Copy link
Contributor Author

Debería decir Cantidad y NO Cantidad de comprimidos, tbn en la Interface debería se cantidad y NO cantComprimidos. Sigue teniendo la restricción de 2 medicamentos máximo.

se cambia nombre de variables de
cantComprimidos > cantidad
en objeto medicamento a registrar, se quita la restricción de 2 medicamentos,
gracias!

@aldoEMatamala aldoEMatamala force-pushed the RUP-395 branch 2 times, most recently from 43f9662 to cb780ad Compare October 22, 2024 17:59
@negro89 negro89 added the dependencies Requiere incorporar otra funcionalidad label Oct 23, 2024
@negro89
Copy link
Contributor

negro89 commented Oct 23, 2024

Dependencia (a la espera de) la tarea Modificaciones molécula RECETA
(No tiene que ver con actualizaciones en este pr, es una referencia para mi)

@aldoEMatamala aldoEMatamala force-pushed the RUP-395 branch 4 times, most recently from b859075 to b8ff12b Compare October 23, 2024 15:53
JuanIRamirez
JuanIRamirez approved these changes Oct 23, 2024
View reviewed changes
Copy link
Contributor

@JuanIRamirez JuanIRamirez left a comment

Choose a reason for hiding this comment

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

OK

@silviroa silviroa removed the dependencies Requiere incorporar otra funcionalidad label Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JuanIRamirez JuanIRamirez JuanIRamirez approved these changes

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

@negro89 negro89 Awaiting requested review from negro89 negro89 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.
6 participants
@aldoEMatamala @MarianoCampetella @MCele @negro89 @JuanIRamirez @silviroa