MISC - Agregar filtro por conceptos en el reporte c2 #2634

MarianoCampetella · 2022-04-26T17:54:20Z

Requerimiento

https://proyectos.andes.gob.ar/browse/MISC-225

Funcionalidad desarrollada

Se agrego un nuevo filtro de cie10.

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

martinebucarey · 2022-04-26T18:11:26Z

USUARIO: mcampetella
BUILD NUMBER: 6286
CYPRESS RUN: 5179
TEST START: 2022-04-26T18:11:12.475Z
TOTAL: 406
SUCCESS: 405
FAIL: 0
SKIPPED: 1

plammel · 2022-04-28T18:32:08Z

src/app/components/reportes/encabezadoReportes.component.ts

+        }
+        if (tipo === 'reporte') {


Suggested change

}

if (tipo === 'reporte') {

} else if (tipo === 'reporte') {

plammel · 2022-04-28T18:32:56Z

src/app/components/reportes/encabezadoReportes.component.ts

+            (value.value) ? this.parametros['cie10'] = this.cie10.codigo : this.parametros['cie10'] = null;
+        }
+        if (tipo === 'reporte') {
+            (this.tipoReportes?.nombre === 'Reporte C2') ? this.reportes = false : this.reportes = true;


Suggested change

(this.tipoReportes?.nombre === 'Reporte C2') ? this.reportes = false : this.reportes = true;

this.reportes = this.tipoReportes?.nombre === 'Reporte C2';

plammel · 2022-04-28T18:33:44Z

src/app/components/reportes/encabezadoReportes.component.ts

@@ -175,4 +182,22 @@ export class EncabezadoReportesComponent implements OnInit {
    add(a, b) {
        return a + b;
    }
+
+    codigoCIE10(event) {
+        if (event && event.query) {


Suggested change

if (event && event.query) {

if (event?.query) {

plammel · 2022-04-28T18:34:19Z

src/app/components/reportes/encabezadoReportes.component.ts

+            };
+            this.cie10Service.get(query).subscribe((datos) => {
+                // mapeamos para mostrar el codigo primero y luego la descripcion
+                datos.map(dato => {


map si obtenemos un nuevo array, sino forEach

Suggested change

datos.map(dato => {

datos.forEach(dato => {

O mejor:

const cie10s = datos.map(dato => ({ id: dato.id, nombre: `(${dato.codigo}) ${dato.nombre}` }))

La explicación seria que en lugar de mandar el array con los objetos completos, retornamos unicamente las tuplas id, nombre que son los datos que necesitamos para desplegar las opciones

martinebucarey · 2022-05-05T12:37:09Z

src/app/components/reportes/encabezadoReportes.html

@@ -18,12 +18,16 @@
                                 (change)="refreshSelection($event,'zonaSanitaria')" [(ngModel)]="zonaSanitaria"
                                 name="zona" [readonly]="totalOrganizaciones">
                    </plex-select>
-                    <plex-select [data]="organizaciones$ | async" label="Organización"
+                    <plex-select tmOrganizaciones label="Organización"


Ojo!! se estas cambiando lo que hiciste acá https://github.com/andes/app/pull/2631/files#

plammel · 2022-05-10T17:47:56Z

src/app/components/reportes/encabezadoReportes.component.ts

@@ -109,6 +110,11 @@ export class EncabezadoReportesComponent implements OnInit {
                this.parametros['zonaSanitaria'] = null;
            }
        }
+        if (tipo === 'cie10') {
+            (value.value) ? this.parametros['cie10'] = this.cie10.codigo : this.parametros['cie10'] = null;


Suggested change

(value.value) ? this.parametros['cie10'] = this.cie10.codigo : this.parametros['cie10'] = null;

this.parametros['cie10'] = value.value ? this.cie10.codigo : null;

plammel · 2022-05-10T18:28:14Z

https://www.loom.com/share/fb6fda500df442c88ab6d1630c79094a
Probando el feature encontré un detalle que no coinciden las codificaciones del resultado con las opciónes de filtro, y no funciona cuando se filtra por dicha codificación. Probablemente sea un tema de datos, pero hay q verificarlo.

MarianoCampetella · 2022-05-11T13:29:22Z

En el listado hay códigos como el A51, A53, etc los cuales no existen tanto en demo como en producción. Por esa razón al momento de buscarlos no devuelve ningún resultado. Hay que consultar si faltan cargar los códigos a la BD.

martinebucarey · 2022-05-13T11:09:19Z

Como el filtro es solo para los reportes C2 solo debería aparecer cuando se selecciona ese reporte, actualmente si cambiamos de reporte sigue apareciendo y se esta enviando como parámetro de la query cuando se realiza la consulta por prestacion, ademas si se cambia de reporte el select queda como readonly y no cambia mas
https://www.loom.com/share/616d3be5f301486e886335373777bb9d

MarianoCampetella · 2022-05-16T13:23:36Z

Como el filtro es solo para los reportes C2 solo debería aparecer cuando se selecciona ese reporte, actualmente si cambiamos de reporte sigue apareciendo y se esta enviando como parámetro de la query cuando se realiza la consulta por prestacion, ademas si se cambia de reporte el select queda como readonly y no cambia mas https://www.loom.com/share/616d3be5f301486e886335373777bb9d

Corregido! ahora cada ves que se cambie el reporte a "consultas por prestacion" el filtro desaparece.

martinebucarey · 2022-06-16T12:24:04Z

Resolver conflictos para subir

martinebucarey · 2022-06-29T12:29:13Z

Resolver conflictos para subir

martinebucarey · 2022-06-29T18:00:03Z

Quedo mal rebaseado

martinebucarey · 2022-06-30T12:45:39Z

USUARIO: mcampetella
BUILD NUMBER: 6720
CYPRESS RUN: 5573
TEST START: 2022-06-30T12:45:24.548Z
TOTAL: 408
SUCCESS: 404
FAIL: 0
SKIPPED: 4

martinebucarey · 2022-06-30T13:48:15Z

https://www.loom.com/share/97985ed5ff114e05829acb0e133469f4

MarianoCampetella · 2022-07-01T12:02:38Z

https://www.loom.com/share/97985ed5ff114e05829acb0e133469f4

Corregido!

# [5.100.0](v5.99.1...v5.100.0) (2022-07-06) ### Bug Fixes * **plan-indiciaciones:** scroll grilla ([#2695](#2695)) ([b6e0a41](b6e0a41)) ### Features * **CITAS:** agrega nombre del dia en listado de agendas ([#2681](#2681)) ([e1c2727](e1c2727)) * **IN-430:** Permitir anular prestaciones de internacion ([#2658](#2658)) ([c1ab483](c1ab483)) * **IN-442:** Labels de resultados en pantalla de censos ([#2672](#2672)) ([aef3a26](aef3a26)) * **INTERNACION:** cambio de lugar botones ([#2692](#2692)) ([599039f](599039f)) * **MISC-225:** Agregar filtro por conceptos en el reporte c2 ([#2634](#2634)) ([a4b9904](a4b9904)) * **RUP:** cambio condicion de required ([#2694](#2694)) ([89ec7f6](89ec7f6))

MarianoCampetella requested review from a team as code owners April 26, 2022 17:54

MarianoCampetella requested review from palita1991 and martinebucarey April 26, 2022 17:54

martinebucarey added the test ok Los test estan ok label Apr 26, 2022

plammel suggested changes Apr 28, 2022

View reviewed changes

MarianoCampetella force-pushed the MISC-225 branch from 031881d to 1f6c825 Compare April 29, 2022 12:39

MarianoCampetella requested a review from plammel April 29, 2022 12:43

github-actions bot added the has_conflicts label May 3, 2022

MarianoCampetella force-pushed the MISC-225 branch from 1f6c825 to f2194a8 Compare May 5, 2022 12:07

github-actions bot removed the has_conflicts label May 5, 2022

martinebucarey requested changes May 5, 2022

View reviewed changes

MarianoCampetella force-pushed the MISC-225 branch from f2194a8 to d15f018 Compare May 5, 2022 17:08

plammel suggested changes May 10, 2022

View reviewed changes

MarianoCampetella force-pushed the MISC-225 branch from d15f018 to 8d2b21a Compare May 11, 2022 13:13

MarianoCampetella requested a review from plammel May 11, 2022 13:30

negro89 approved these changes May 11, 2022

View reviewed changes

negro89 requested a review from martinebucarey May 11, 2022 14:56

MarianoCampetella force-pushed the MISC-225 branch from 8d2b21a to 68c8653 Compare May 16, 2022 13:21

martinebucarey approved these changes Jun 10, 2022

View reviewed changes

github-actions bot added the has_conflicts label Jun 13, 2022

github-actions bot removed the has_conflicts label Jun 29, 2022

martinebucarey self-requested a review June 29, 2022 17:59

palita1991 force-pushed the MISC-225 branch 2 times, most recently from 68c8653 to 1361e1f Compare June 30, 2022 12:06

martinebucarey added test ok Los test estan ok and removed test ok Los test estan ok labels Jun 30, 2022

martinebucarey approved these changes Jul 1, 2022

View reviewed changes

feat(MISC-225): Agregar filtro por conceptos en el reporte c2

8f7a7c2

martinebucarey force-pushed the MISC-225 branch from 1361e1f to 8f7a7c2 Compare July 1, 2022 13:39

martinebucarey merged commit a4b9904 into master Jul 1, 2022

martinebucarey deleted the MISC-225 branch July 1, 2022 13:51

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

MISC - Agregar filtro por conceptos en el reporte c2 #2634

MISC - Agregar filtro por conceptos en el reporte c2 #2634

MarianoCampetella commented Apr 26, 2022

martinebucarey commented Apr 26, 2022

plammel Apr 28, 2022

MarianoCampetella Apr 29, 2022

plammel Apr 28, 2022

MarianoCampetella Apr 29, 2022

plammel Apr 28, 2022

MarianoCampetella Apr 29, 2022

plammel Apr 28, 2022

plammel Apr 28, 2022

MarianoCampetella Apr 29, 2022

martinebucarey May 5, 2022

MarianoCampetella May 5, 2022

plammel May 10, 2022

MarianoCampetella May 11, 2022

plammel commented May 10, 2022

MarianoCampetella commented May 11, 2022

martinebucarey commented May 13, 2022

MarianoCampetella commented May 16, 2022

martinebucarey commented Jun 16, 2022

martinebucarey commented Jun 29, 2022

martinebucarey commented Jun 29, 2022

martinebucarey commented Jun 30, 2022

martinebucarey commented Jun 30, 2022

MarianoCampetella commented Jul 1, 2022

	(this.tipoReportes?.nombre === 'Reporte C2') ? this.reportes = false : this.reportes = true;
	this.reportes = this.tipoReportes?.nombre === 'Reporte C2';

	(value.value) ? this.parametros['cie10'] = this.cie10.codigo : this.parametros['cie10'] = null;
	this.parametros['cie10'] = value.value ? this.cie10.codigo : null;

MISC - Agregar filtro por conceptos en el reporte c2 #2634

MISC - Agregar filtro por conceptos en el reporte c2 #2634

Conversation

MarianoCampetella commented Apr 26, 2022

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

martinebucarey commented Apr 26, 2022

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

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

Choose a reason for hiding this comment

plammel commented May 10, 2022

MarianoCampetella commented May 11, 2022

martinebucarey commented May 13, 2022

MarianoCampetella commented May 16, 2022

martinebucarey commented Jun 16, 2022

martinebucarey commented Jun 29, 2022

martinebucarey commented Jun 29, 2022

martinebucarey commented Jun 30, 2022

martinebucarey commented Jun 30, 2022

MarianoCampetella commented Jul 1, 2022