CIT: carga masiva de pacientes #2526

plammel · 2021-11-18T17:46:46Z

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-134
https://proyectos.andes.gob.ar/browse/CIT-153

Funcionalidad desarrollada

Permite carga masiva de pacientes por consulta

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

Requiere actualizaciones en andesMicroservices

Si
No

martinebucarey

Cuando asignas los turnos, queda un disponible medio raro en los turnos
Por otro lado vamos a validar si con el permiso y que la agenda tenga pacientes simultáneos alcanza para que ya se pueda cargar masivamente. O que pueda llegar a haber agendas sin pacientes simultáneos que también se quieran cargar.

martinebucarey · 2021-11-19T12:20:12Z

src/app/components/turnos/gestor-agendas/operaciones-agenda/botones-agenda.component.ts

@@ -138,6 +139,7 @@ export class BotonesAgendaComponent implements OnInit {
        const puedeBorrar = this.auth.getPermissions('turnos:agenda:puedeBorrar:').length > 0;
        const puedeRevisar = this.auth.getPermissions('turnos:agenda:puedeRevision:').length > 0;
        const puedeNota = this.auth.getPermissions('turnos:agenda:puedeNota:').length > 0;
+        const puedeCargar = true;


Suggested change

const puedeCargar = true;

const puedeCargar = this.auth.getPermissions('turnos:agenda:asignacionMasiva:').length > 0;

src/app/modules/visualizacion-informacion/components/bi-queries/bi-queries.component.html

liquid36 · 2021-12-07T13:56:34Z

src/app/components/turnos/gestor-agendas/gestor-agendas.component.ts

+        }));
+
+        let turnos = [];
+        this.agendasSeleccionadas[0].bloques.forEach(b => {


Para analizar, si no hay lugar en la agenda no metería la mitad de los pacientes. Arrojaría un error.

Si la agenda es dinámica que pasa?

liquid36

Queda pendiente sacar las queries desde los conceptos turneables.

src/app/components/turnos/gestor-agendas/gestor-agendas.html

liquid36 · 2021-12-07T14:00:05Z

src/app/components/turnos/gestor-agendas/gestor-agendas.html

@@ -190,6 +191,12 @@
                        </suspender-agenda>
                    </div>
                </plex-box>
+                <plex-box *ngIf="enableQueries">
+                    <app-bi-queries type="inscriptos-vacunacion">


Esta prohibido meter un plex-layout dentro de otro plex-layout. Acostumbrense a refactorizar componentes para que sean más reutilizables intramodulos.

La componente de bi-query debería quedar separada en dos. Una es la componente del routing y otra el motor que genera los argumentos dinámicamente.

martinebucarey

No entiendo la separación de componentes bi, hay dos exactamente iguales y solo cambia la vista que uno llama al otro. Se hace por algo en especial así??

martinebucarey · 2021-12-15T12:48:36Z

src/app/modules/visualizacion-informacion/components/bi-queries/bi-queries.component.ts

@@ -15,6 +15,8 @@ import { IZonaSanitaria } from 'src/app/interfaces/IZonaSanitaria';
    styleUrls: ['./bi-queries.component.scss']
 })
 export class BiQueriesComponent implements OnInit {
+    @Input() type;


Teniendo el array de queries, el type seguiría siendo necesario?? yo pensaba controlar lo que se hacia por el type, ahora por el array

removido el type

martinebucarey · 2021-12-15T13:40:40Z

src/app/modules/visualizacion-informacion/components/bi-queries/bi-queries.component.html

@@ -1,9 +1,10 @@
 <plex-layout>
    <plex-layout-main>
        <plex-title titulo="Ejecutar consultas">
-            <plex-button size="sm" type="success" (click)="descargar()" [disabled]="form.invalid">
+            <plex-button *ngIf="!type" size="sm" type="success" (click)="descargar()" [disabled]="form.invalid">


Suggested change

<plex-button *ngIf="!type" size="sm" type="success" (click)="descargar()" [disabled]="form.invalid">

<plex-button *ngIf="!queries.length" size="sm" type="success" (click)="descargar()" [disabled]="form.invalid">

martinebucarey · 2021-12-15T13:41:01Z

src/app/modules/visualizacion-informacion/components/bi-queries/bi-queries.component.html

@@ -67,9 +68,9 @@
            </ng-container>
        </form>
        <div justify="center" class="h-75">
-            <plex-label titulo="Comience seleccionando una consulta" direction="column"
+            <plex-label *ngIf="!type" titulo="Comience seleccionando una consulta" direction="column"


Suggested change

<plex-label *ngIf="!type" titulo="Comience seleccionando una consulta" direction="column"

<plex-label *ngIf="!queries.length" titulo="Comience seleccionando una consulta" direction="column"

liquid36 · 2021-12-22T12:31:12Z

USUARIO: plammel
BUILD NUMBER: 5704
CYPRESS RUN: 4628
TEST START: 2021-12-22T12:31:01.332Z
TOTAL: 404
SUCCESS: 401
FAIL: 2
SKIPPED: 1

martinebucarey · 2021-12-22T18:08:26Z

src/app/components/turnos/gestor-agendas/gestor-agendas.html

@@ -190,6 +191,12 @@
                        </suspender-agenda>
                    </div>
                </plex-box>
+                <plex-box *ngIf="enableQueries">
+                    <app-bi-queries [queries]="queries">
+                        <plex-button size="sm" type="danger" (click)="cerrarCargarPacientes()">Cerrara</plex-button>


Suggested change

<plex-button size="sm" type="danger" (click)="cerrarCargarPacientes()">Cerrara</plex-button>

<plex-button size="sm" type="danger" (click)="cerrarCargarPacientes()">Cerrar</plex-button>

liquid36 · 2021-12-22T18:13:11Z

USUARIO: plammel
BUILD NUMBER: 5709
CYPRESS RUN: 4633
TEST START: 2021-12-22T18:01:49.058Z
TOTAL: 404
SUCCESS: 384
FAIL: 19
SKIPPED: 1

martinebucarey

Revisar que hay casos donde no se estan setenado los id de los pacientes en el turno y por ese motivo se asignan los turnos pero no se muestran en el sidebar

plammel · 2022-02-15T15:08:59Z

Por alguna razon esta guardando vacio el array de queries cuando creo la agenda, adjunto las imagenes de lo que tiene la agenda y como esta en la base el concepto, lo probe en demo y local y me pasa lo mismo

updated!

liquid36 · 2022-02-15T17:11:21Z

USUARIO: martinebucarey
BUILD NUMBER: 5931
CYPRESS RUN: 4848
TEST START: 2022-02-15T17:11:09.922Z
TOTAL: 404
SUCCESS: 402
FAIL: 1
SKIPPED: 1

liquid36 · 2022-02-15T17:20:58Z

USUARIO: martinebucarey
BUILD NUMBER: 5932
CYPRESS RUN: 4849
TEST START: 2022-02-15T17:20:46.572Z
TOTAL: 404
SUCCESS: 402
FAIL: 1
SKIPPED: 1

martinebucarey

En el caso de tener permisos para todas las queries, el componente se rompe porque quiere mandar un _id=*

liquid36 · 2022-02-17T15:54:13Z

USUARIO: martinebucarey
BUILD NUMBER: 5940
CYPRESS RUN: 4857
TEST START: 2022-02-17T15:53:29.556Z
TOTAL: 343
SUCCESS: 331
FAIL: 11
SKIPPED: 1

liquid36 · 2022-02-17T16:03:30Z

USUARIO: martinebucarey
BUILD NUMBER: 5941
CYPRESS RUN: 4858
TEST START: 2022-02-17T16:03:17.174Z
TOTAL: 406
SUCCESS: 404
FAIL: 1
SKIPPED: 1

liquid36 · 2022-02-17T16:50:10Z

USUARIO: martinebucarey
BUILD NUMBER: 5942
CYPRESS RUN: 4859
TEST START: 2022-02-17T16:49:59.340Z
TOTAL: 406
SUCCESS: 405
FAIL: 0
SKIPPED: 1

# [5.85.0](v5.84.0...v5.85.0) (2022-02-23) ### Features * **citas:** carga masiva ([#2526](#2526)) ([4543322](4543322)) * **ficha:** nuevos tipos de confirmacion ([#2586](#2586)) ([aba5629](aba5629)) * **vacunas:** nueva pantalla verificar estado ([#2584](#2584)) ([c22e1e5](c22e1e5))

plammel requested review from a team as code owners November 18, 2021 17:46

plammel requested review from MCele and negro89 November 18, 2021 17:46

plammel force-pushed the CIT-134b branch from 8ec84b2 to c9931ba Compare November 18, 2021 18:12

martinebucarey requested changes Nov 19, 2021

View reviewed changes

plammel force-pushed the CIT-134b branch from c9931ba to b57004d Compare November 24, 2021 14:13

plammel requested a review from martinebucarey November 24, 2021 14:13

liquid36 reviewed Dec 7, 2021

View reviewed changes

liquid36 requested changes Dec 7, 2021

View reviewed changes

liquid36 added the changes requested Se solicitaron cambios label Dec 7, 2021

plammel force-pushed the CIT-134b branch 2 times, most recently from f9bf0da to 4e467f6 Compare December 14, 2021 15:38

plammel requested a review from liquid36 December 14, 2021 16:47

martinebucarey reviewed Dec 15, 2021

View reviewed changes

plammel force-pushed the CIT-134b branch from 4e467f6 to 294f4f4 Compare December 15, 2021 13:35

plammel requested a review from martinebucarey December 15, 2021 13:36

martinebucarey requested changes Dec 15, 2021

View reviewed changes

plammel force-pushed the CIT-134b branch from 294f4f4 to 6aa90c4 Compare December 21, 2021 14:49

plammel requested a review from martinebucarey December 21, 2021 14:49

plammel force-pushed the CIT-134b branch from 6aa90c4 to a875099 Compare December 21, 2021 15:48

plammel removed the changes requested Se solicitaron cambios label Dec 22, 2021

liquid36 added the test fail label Dec 22, 2021

martinebucarey requested changes Dec 22, 2021

View reviewed changes

liquid36 added test fail and removed test fail labels Dec 22, 2021

martinebucarey requested changes Dec 22, 2021

View reviewed changes

plammel force-pushed the CIT-134b branch 2 times, most recently from 3b14179 to ae64c84 Compare February 15, 2022 14:47

liquid36 added test fail and removed test fail labels Feb 15, 2022

martinebucarey requested changes Feb 15, 2022

View reviewed changes

plammel removed the has_conflicts label Feb 16, 2022

plammel force-pushed the CIT-134b branch 2 times, most recently from 15a6404 to 556fe44 Compare February 16, 2022 17:59

plammel requested a review from martinebucarey February 16, 2022 18:22

plammel force-pushed the CIT-134b branch from 556fe44 to 5f96244 Compare February 17, 2022 14:06

feat(citas): carga masiva

d71be98

martinebucarey force-pushed the CIT-134b branch from 5f96244 to d71be98 Compare February 17, 2022 15:41

liquid36 added test fail and removed test fail labels Feb 17, 2022

liquid36 added test ok Los test estan ok and removed test fail labels Feb 17, 2022

martinebucarey approved these changes Feb 17, 2022

View reviewed changes

martinebucarey merged commit 4543322 into master Feb 17, 2022

martinebucarey deleted the CIT-134b branch February 17, 2022 16:52

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CIT: carga masiva de pacientes #2526

CIT: carga masiva de pacientes #2526

plammel commented Nov 18, 2021 •

edited

Loading

martinebucarey left a comment

martinebucarey Nov 19, 2021

liquid36 Dec 7, 2021

liquid36 left a comment

liquid36 Dec 7, 2021

liquid36 Dec 7, 2021

martinebucarey left a comment

martinebucarey Dec 15, 2021

plammel Dec 15, 2021

martinebucarey Dec 15, 2021

martinebucarey Dec 15, 2021

plammel Dec 21, 2021

liquid36 commented Dec 22, 2021

martinebucarey Dec 22, 2021

liquid36 commented Dec 22, 2021

martinebucarey left a comment

plammel commented Feb 15, 2022

liquid36 commented Feb 15, 2022

liquid36 commented Feb 15, 2022

martinebucarey left a comment

liquid36 commented Feb 17, 2022

liquid36 commented Feb 17, 2022

liquid36 commented Feb 17, 2022

	const puedeCargar = true;
	const puedeCargar = this.auth.getPermissions('turnos:agenda:asignacionMasiva:').length > 0;

	<plex-button *ngIf="!type" size="sm" type="success" (click)="descargar()" [disabled]="form.invalid">
	<plex-button *ngIf="!queries.length" size="sm" type="success" (click)="descargar()" [disabled]="form.invalid">

	<plex-label *ngIf="!type" titulo="Comience seleccionando una consulta" direction="column"
	<plex-label *ngIf="!queries.length" titulo="Comience seleccionando una consulta" direction="column"

	<plex-button size="sm" type="danger" (click)="cerrarCargarPacientes()">Cerrara</plex-button>
	<plex-button size="sm" type="danger" (click)="cerrarCargarPacientes()">Cerrar</plex-button>

CIT: carga masiva de pacientes #2526

CIT: carga masiva de pacientes #2526

Conversation

plammel commented Nov 18, 2021 • edited Loading

Requerimiento

Funcionalidad desarrollada

UserStory llegó a completarse

Requiere actualizaciones en la base de datos

Requiere actualizaciones en la API

Requiere actualizaciones en andes-test-integracion

Requiere actualizaciones en andesMicroservices

martinebucarey left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

liquid36 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

martinebucarey left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

liquid36 commented Dec 22, 2021

Choose a reason for hiding this comment

liquid36 commented Dec 22, 2021

martinebucarey left a comment

Choose a reason for hiding this comment

plammel commented Feb 15, 2022

liquid36 commented Feb 15, 2022

liquid36 commented Feb 15, 2022

martinebucarey left a comment

Choose a reason for hiding this comment

liquid36 commented Feb 17, 2022

liquid36 commented Feb 17, 2022

liquid36 commented Feb 17, 2022

plammel commented Nov 18, 2021 •

edited

Loading