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

Epidemiolgía - Clasificacion final caso asintomatico #2482

Merged
merged 1 commit into from
Oct 22, 2021
Merged

Epidemiolgía - Clasificacion final caso asintomatico #2482

merged 1 commit into from
Oct 22, 2021

Conversation

martinebucarey
Copy link
Contributor

Requerimiento

https://proyectos.andes.gob.ar/browse/EP-146

Funcionalidad desarrollada

  1. Al clasificar el caso en la primer sección como asintomático estudiado en situaciones especiales, en la clasificación final se debe setear el campo como Caso Asintomático y dejar seleccionado PCR como tipo de confirmación.

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

@liquid36
Copy link
Contributor

USUARIO: martinebucarey
BUILD NUMBER: 5213
CYPRESS RUN: 4160
TEST START: 2021-09-30T15:32:39.754Z
TOTAL: 397
SUCCESS: 395
FAIL: 1
SKIPPED: 1

@liquid36
Copy link
Contributor

USUARIO: martinebucarey
BUILD NUMBER: 5214
CYPRESS RUN: 4161
TEST START: 2021-09-30T16:52:04.777Z
TOTAL: 401
SUCCESS: 400
FAIL: 0
SKIPPED: 1

@liquid36 liquid36 added test ok Los test estan ok and removed test fail labels Sep 30, 2021
@@ -250,7 +250,7 @@ export class FichaEpidemiologicaCrudComponent implements OnInit, OnChanges {
}

ngOnInit(): void {
// Pregunta por el permiso de huds para el caso en el que se visualiza una ficha desde la huds y no tiene permisos de epidemiologia
// Pregunta por el permiso de huds para el caso en el que se visualiza una ficha desde la huds y no tiene permisos de epidemiologia
Copy link
Contributor

Choose a reason for hiding this comment

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

Ojo con estas cosas, deberían existir varios componentes para no traer estas comfusiones. DEsde la HUDS ese código no debería traer problemas.

Los ruteos deben ocurrir solo en las componentes raices del router.

Lo dejo como comentario para futuros cambios y nuevas pantallas.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Lo que decís es que yo no debería preguntar por el permiso de la huds acá no?? osea me debería rebotar el componente de la huds si no tengo ese permiso y no este que no tiene nada que ver.

@kroce kroce removed the request for review from condorpiedra October 22, 2021 13:55
@kroce kroce merged commit 710f636 into master Oct 22, 2021
@kroce kroce deleted the EP-146 branch October 22, 2021 15:29
liquid36 pushed a commit that referenced this pull request Oct 27, 2021
# [5.70.0](v5.69.0...v5.70.0) (2021-10-27)

### Bug Fixes

* **novedades:** se alinean títulos de items ([#2501](#2501)) ([f8f7127](f8f7127))

### Features

* **bi:** filtros por organización y zona en dashboard de citas ([#2502](#2502)) ([c2d6d3e](c2d6d3e))
* **ficha:** caso asintomatico ([#2482](#2482)) ([710f636](710f636))
* **GDU:** control para cuando el profesional no tiene numero de matricula ([#2496](#2496)) ([4013e33](4013e33))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants