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.

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

RUP - Receta : modificar molécula #3005

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

RUP - Receta : modificar molécula #3005

wants to merge 1 commit into from

Conversation

aldoEMatamala
Copy link
Contributor

@aldoEMatamala aldoEMatamala commented May 3, 2024

Requerimiento

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

Funcionalidad desarrollada

  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

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

@aldoEMatamala aldoEMatamala requested review from a team as code owners May 3, 2024 14:12
@aldoEMatamala
Copy link
Contributor Author

imagen

@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

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes done En pausa Temporalmente pausado test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants