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 - Modifica controles en moléculas #3092

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

RUP - Modifica controles en moléculas #3092

wants to merge 1 commit into from

Conversation

ma7payne
Copy link
Contributor

Requerimiento

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

Funcionalidad desarrollada

  1. Modifica controles para molécula "Medir la altura del paciente"
  2. Modifica controles para molécula "Presión sistólica"
  3. Modifica controles para molécula "Presión diastólicae"

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

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.

Funcionan bien las validaciones!, revisar la tensión sistolica minima

</plex-int>
<form #form="ngForm">
<plex-int *ngIf="!soloValores" placeholder="Ingrese un valor entre 20 y 200" [(ngModel)]="registro.valor"
name="tensionSistolica" label="Tensión Sistólica" suffix="mmHG" min="20" max="200" required="true"
Copy link
Contributor

Choose a reason for hiding this comment

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

Funciona bien la validación valores al guardar el elemento Control de signos vitales y la talla, lo unico que el valor minimo de la Tension sistolica sigue siendo 20 en vez de 30 como dice el requrimiento al igual que la ayuda del placeholder.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Listo el cambio!

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.

2 participants