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

IN - Nueva opción 'otro' en tipo de egreso y organización destino #2310

Merged
merged 4 commits into from
Jun 9, 2021
Merged

IN - Nueva opción 'otro' en tipo de egreso y organización destino #2310

merged 4 commits into from
Jun 9, 2021

Conversation

negro89
Copy link
Contributor

@negro89 negro89 commented Jun 2, 2021 •
edited
Loading

Requerimiento

https://proyectos.andes.gob.ar/browse/IN-288
https://proyectos.andes.gob.ar/browse/IN-262

Funcionalidad desarrollada

  1. Se agrega opción 'otro' a a la constante de internación 'listaTipoegreso'.
  2. Nuevo campo de texto libre para ingresar 'otra' organización destino en caso de egreso por traslado.

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

@negro89 negro89 requested review from a team as code owners June 2, 2021 14:48
@negro89 negro89 requested a review from a team as a code owner June 2, 2021 16:37
@negro89 negro89 changed the title IN - Nuevo tipo de egreso 'otro' IN - Tipo de egreso y organización destino 'otro' Jun 2, 2021
@negro89 negro89 changed the title IN - Tipo de egreso y organización destino 'otro' IN - Nueva opción 'otro' en tipo de egreso y organización destino Jun 2, 2021
@liquid36
Copy link
Contributor

liquid36 commented Jun 2, 2021

USUARIO: lmlagos
BUILD NUMBER: 4241
CYPRESS RUN: 3267
TEST START: 2021-06-02T16:53:18.605Z
TOTAL: 395
SUCCESS: 395
FAIL: 0
SKIPPED: 0

@liquid36 liquid36 added the test ok Los test estan ok label Jun 2, 2021
sychus
sychus suggested changes Jun 2, 2021
View reviewed changes
Copy link

@sychus sychus left a comment

Choose a reason for hiding this comment

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

En el caso del COM están todas las organizaciones que es posible trasladar. El campo organización tiene un flag para identificar las que aceptan traslado.
Por otro lado, no está bueno que quede con un texto libre ya q luego no nos permitirá hacer ningún tipo de estadística.
Hay que ver bien el requerimiento (considero que no está bien), ya que todos los traslados se solicitan al COM y son ellos quienes deciden a que institución se traslada.

liquid36
liquid36 requested changes Jun 2, 2021
View reviewed changes
Copy link
Contributor

@liquid36 liquid36 left a comment

Choose a reason for hiding this comment

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

Hay incosistencia del tipo de dato de la variable UnidadOrganizativaDestino. En ambos casos debería ser un objeto con id y nombre. En el caso de otro el id queda vacío.

Al editar un egreso, se pierde el estado de otro y lo que escribí.

De Yapa agregar la info a la componente informe-egreso

@negro89
Copy link
Contributor Author

negro89 commented Jun 3, 2021

Creo que tiene mucho sentido lo que dice @sychus. Quizás le haga falta un poco mas de análisis a esta tarea.
@liquid36 no se entendrió lo de la yapa.

@sychus
Copy link

sychus commented Jun 3, 2021

Creo que tiene mucho sentido lo que dice @sychus. Quizás le haga falta un poco mas de análisis a esta tarea.
@liquid36 no se entendrió lo de la yapa.

Ayer hable con la gente del COM sobre este tema, 2 temas:

  1. Todo lo que es derivación COVID es como comenté arriba, por lo tanto hay q darle una vuelta de rosca y ver bien el requerimiento tal vez es hora de enganchar con COM.
  2. Todo lo que es NO COVID hoy no es posible absorverlo por el com debido a que no hay RRHH, pero a futuro todas las derivaciones van a pasar por COM. Sin bien el módulo de COM está preparado para cualquier tipo de derivación hoy lo que es NO COVID va por otra vía, tenemos que ver ahí que es lo mejor para poder hacerlo ordenado.

@liquid36
Copy link
Contributor

liquid36 commented Jun 3, 2021

Quedo charlado con @silviroa. Es un REQ que están pidiendo hace rato María de Estadistica.
Además el COM no tiene todos los traslado, no me parece prudente imponer bloqueos a la carga de datos en este momento . Si sería ideal mostrar datos de la derivación en el mapa de camas y a futuro sugerir que se fue por un traslado.

La yapa es que al ver el resumen del egreso no se esta mostrando los datos del traslado. Esta es la componente: https://github.com/andes/app/blob/master/src/app/apps/rup/mapa-camas/sidebar/egreso/informe-egreso.component.ts#L13

@negro89 negro89 requested review from liquid36 and sychus June 3, 2021 18:45
@liquid36
Copy link
Contributor

liquid36 commented Jun 9, 2021

@negro89 el caso de editar un egreso no había sido contemplado!

@liquid36 liquid36 merged commit 8b0d11e into master Jun 9, 2021
@liquid36 liquid36 deleted the IN-288 branch June 9, 2021 13:39
liquid36 pushed a commit that referenced this pull request Jun 9, 2021
@semantic-release-bot
chore(release): 5.50.0 [skip ci]
b9b98f3 
# [5.50.0](v5.49.0...v5.50.0) (2021-06-09)

### Bug Fixes

* **inscripcion:** agrega grupo sin FR ([#2311](#2311)) ([a431896](a431896))
* **mapa-camas:** corrige dias de estada totales en censo mensual ([#2315](#2315)) ([338b60e](338b60e))

### Features

* **com:** se cambia lo de switchmap y lo de los params ([05f79b7](05f79b7))
* **COM:** asignar origen desde org COM ([89cc173](89cc173))
* **ficha:** field localidades ([f405f22](f405f22))
* **ficha-epidemiogica:** check dias ([fe23442](fe23442))
* **inscripcion:** agrega control de edad por reglas ([#2309](#2309)) ([862afba](862afba))
* **login:** se implementa v0.1 ([#2238](#2238)) ([a01a4d7](a01a4d7))
* **mapa-camas:**  seleccion de otra organizacion en traslado ([#2310](#2310)) ([8b0d11e](8b0d11e))
* **mapa-camas:** guarda unidad organizativa en prestacion ([#2283](#2283)) ([ea930c6](ea930c6))
* **mapa-camas:** plex-table en listado de internacion ([#2317](#2317)) ([43eaa8a](43eaa8a))
* **mapa-camas:** sincronizar plex-datetime en resumen internacion ([#2307](#2307)) ([520cbc6](520cbc6))
* **mapa-camsa:** agrega fecha de ingreso en censo diario ([#2308](#2308)) ([79fdbe3](79fdbe3))
* **vac:** desvincular paciente de inscripción ([#2277](#2277)) ([2d4d587](2d4d587))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liquid36 liquid36 liquid36 approved these changes

@condorpiedra condorpiedra Awaiting requested review from condorpiedra

@plammel plammel Awaiting requested review from plammel

@sychus sychus Awaiting requested review from sychus

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.
3 participants
@negro89 @liquid36 @sychus