CIT - Nuevo submódulo Auditoría de Agendas #2882

JuanIRamirez · 2023-06-22T14:05:01Z

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-227

Funcionalidad desarrollada

Nuevo submódulo de Auditoría de Agendas.
Nuevo componente: auditoria-agendas.component.ts
Modificados: gestor-agendas, fuera-agenda, sobreturno, revision-agenda, agregar-paciente.
Se separa la Auditoría del Gestor de Agendas. para separar acceso según permisos.

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

Agregar este objeto al arreglo 'submodulos' del documento 'Citas' en colección 'modulos'
{ nombre:'CITAS'}

        {
            "activo" : true,
            "orden" : 0,
            "icono" : "table-search",
            "nombre" : "Auditoría de Agendas",
            "descripcion" : "Auditoría de turnos y Prestaciones",
            "class" : "citas audit",
            "color" : "#00a8e0",
            "linkAcceso" : "citas/auditoria_agendas",
            "permisos" : [ 
                "turnos:auditoriaAgenda:?"
            ]
        }

AgosLizzi

Juan, me da la impresión que muchas cosas que están implementadas en el mockup no se tuvieron en cuenta y en consecuencia se usaron los recursos de los módulos viejos. Te voy detallando algunas cosas. Pero mi recomendación en general, es que revises nuevamente los mockups:

Te dejo la comparación de imagenes de implementación/ imagenes: Estaría faltando icono, corregir ordenamiento y el tamaño de icono para cerrar es demasido grande, lo achicaria.
Los iconos están demasiado separados. De yapa agregaría el icono que se presenta en el mockup que ahora sabemos que esta disponible en andesfont
Esta tabla, es la que esta usada sobre las funciones viejas. Además muchos badge no estan contemplados, como el de "turno libre". Como referencia, la que esta reflajada en el mockup es la misma que se ve en el gestor de agendas.
Los iconos sobre la asistencia no están contemplados en el mockup, y faltan las tildes sobre las palabras
/
Como sumatoria del punto anterior, una vez que doy asistencia en el mockup se utiliza un badge para dar esa indicación a la persona usuaria. En esta implementación no veo dicha acción y por lo contrario, tengo nuevamente un botón para generar asistencia. Ejemplo:
Aca también. Revisar como están los botones en el mockup
La codificación esta planteada como estaba en el módulo anterior, diferente al mockup. También se ve reflejado un botón "volver" que no se de donde sale.

Nuevamente, no se reflejan los componentes del mockup.
Por qué ese contexto del checkbox?
El color del módulo, no es el que esta contemplado en el mockup. No debería ser el mismo que el gestor de agendas, por eso se uso un color diferenciador.
Botones en diferente lugar que en el mockup
Qué sucedió aca? ... a la palabra revisión le agregaría la tilde que le falta

MCele

Juan en general el funcionamiento está bastante bien, te dejo algunas observaciones de lo que noté para revisar.
Detalles visuales:

Cuando en el tab de fuera de agenda no se encuentren datos a mostrar no debería visualizarse el listado vacío, si no un mensaje notificando que no se encontraron prestaciones para los filtros seleccionados.
El filtro de Espacios físicos ya no es necesario en esta pantalla.
Cuando se cambia algún filtro desde fuera de agendas, si se elige el tab de agendas no se actualiza el listado con los nuevos filtros.
El filtro (check) "ver prestaciones ya auditadas" puede considerarse el filtro de estado para las fuera de agendas.
Detalles no visuales:
No funciona el scroll infinito en fuera de agenda.

silviroa · 2023-06-28T18:59:46Z

USUARIO: jramirez
BUILD NUMBER: 9127
CYPRESS RUN: 8274
TEST START: 2023-06-28T18:59:33.067Z
TOTAL: 403
SUCCESS: 393
FAIL: 8
SKIPPED: 2

silviroa · 2023-06-28T19:20:20Z

USUARIO: lmlagos
BUILD NUMBER: 9129
CYPRESS RUN: 8276
TEST START: 2023-06-28T19:20:08.992Z
TOTAL: 403
SUCCESS: 393
FAIL: 8
SKIPPED: 2

MCele · 2023-06-29T20:04:38Z

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

-import { EspacioFisicoService } from './../../../services/turnos/espacio-fisico.service';
-import * as enumerado from './../enums';
+import { IAgenda } from '../../../interfaces/turnos/IAgenda';
+import { ProfesionalService } from '../../../services/profesional.service';


me parece que no son necesarios los cambios en las rutas de los imports

Me daba error de no encontrado. por eso las cambié.

MCele · 2023-06-29T20:18:43Z

Revisar el commit de este pr, parece haberse quedado con el de otro PR

silviroa · 2023-06-30T17:56:11Z

USUARIO: jramirez
BUILD NUMBER: 9146
CYPRESS RUN: 8293
TEST START: 2023-06-30T17:53:02.435Z
TOTAL: 403
SUCCESS: 392
FAIL: 9
SKIPPED: 2

silviroa · 2023-06-30T18:33:29Z

USUARIO: jramirez
BUILD NUMBER: 9148
CYPRESS RUN: 8295
TEST START: 2023-06-30T18:33:16.757Z
TOTAL: 403
SUCCESS: 393
FAIL: 8
SKIPPED: 2

--------- Co-authored-by: juan I. Ramirez

silviroa · 2024-01-17T14:03:56Z

USUARIO: mcampetella
BUILD NUMBER: 10294
CYPRESS RUN: 9401
TEST START: 2024-01-17T14:02:38.904Z
TOTAL: 91
SUCCESS: 0
FAIL: 91
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

--------- Co-authored-by: juan I. Ramirez

silviroa · 2024-01-25T13:13:12Z

USUARIO: lmlagos
BUILD NUMBER: 10330
CYPRESS RUN: 9437
TEST START: 2024-01-25T13:08:53.584Z
TOTAL: 91
SUCCESS: 0
FAIL: 91
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

silviroa · 2024-01-25T13:25:19Z

USUARIO: lmlagos
BUILD NUMBER: 10331
CYPRESS RUN: 9438
TEST START: 2024-01-25T13:24:07.420Z
TOTAL: 336
SUCCESS: 327
FAIL: 6
SKIPPED: 3

silviroa · 2024-01-25T14:05:58Z

USUARIO: lmlagos
BUILD NUMBER: 10333
CYPRESS RUN: 9440
TEST START: 2024-01-25T14:04:43.358Z
TOTAL: 365
SUCCESS: 357
FAIL: 5
SKIPPED: 3

silviroa · 2024-01-25T18:16:04Z

USUARIO: lmlagos
BUILD NUMBER: 10340
CYPRESS RUN: 9447
TEST START: 2024-01-25T18:15:43.298Z
TOTAL: 403
SUCCESS: 401
FAIL: -1
SKIPPED: 3

negro89 · 2024-02-29T17:07:47Z

Se publica en produccion el 10 de marzo.

JuanIRamirez requested review from a team as code owners June 22, 2023 14:05

JuanIRamirez requested review from ma7payne and AgosLizzi June 22, 2023 14:05

JuanIRamirez force-pushed the CIT-227 branch 3 times, most recently from f22110d to c26e6aa Compare June 23, 2023 13:38

AgosLizzi requested changes Jun 26, 2023

View reviewed changes

MCele requested changes Jun 26, 2023

View reviewed changes

negro89 added the changes requested Se solicitaron cambios label Jun 28, 2023

silviroa added the test fail label Jun 28, 2023

silviroa added test fail and removed test fail labels Jun 28, 2023

MCele reviewed Jun 29, 2023

View reviewed changes

JuanIRamirez force-pushed the CIT-227 branch 2 times, most recently from 8cef01c to c0e5b4b Compare June 30, 2023 13:20

silviroa added test fail and removed test fail labels Jun 30, 2023

JuanIRamirez force-pushed the CIT-227 branch from c0e5b4b to a9e1711 Compare July 3, 2023 13:20

JuanIRamirez changed the title ~~Feat(CIT-227) Nuevo submódulo Auditoría de Agendas~~ CIT - Nuevo submódulo Auditoría de Agendas Jul 5, 2023

JuanIRamirez force-pushed the CIT-227 branch from a9e1711 to b8bd6c7 Compare July 6, 2023 12:41

JuanIRamirez requested a review from MCele July 6, 2023 12:55

MarianoCampetella added the Aprobado label Dec 22, 2023

AgosLizzi approved these changes Dec 22, 2023

View reviewed changes

negro89 requested a review from silviroa December 22, 2023 12:26

github-actions bot added the has_conflicts label Dec 22, 2023

MarianoCampetella pushed a commit that referenced this pull request Jan 17, 2024

CIT-Nuevo submódulo Auditoría de Agendas #2882

112693d

--------- Co-authored-by: juan I. Ramirez

MarianoCampetella force-pushed the CIT-227 branch from b10cd14 to 6d84c76 Compare January 17, 2024 13:30

MarianoCampetella removed the has_conflicts label Jan 17, 2024

silviroa added test fail and removed test ok Los test estan ok labels Jan 17, 2024

plammel and others added 2 commits January 25, 2024 09:26

CIT-Nuevo submódulo Auditoría de Agendas #2882

5014d0c

--------- Co-authored-by: juan I. Ramirez

Agrega toast de confirmación/error en operaciones de codificación

72e300c

negro89 force-pushed the CIT-227 branch from 6d84c76 to 72e300c Compare January 25, 2024 12:27

silviroa added test fail and removed test fail labels Jan 25, 2024

silviroa added test ok Los test estan ok and removed test fail labels Jan 25, 2024

negro89 added muy prioritario Tarea con prioridad muy alta and removed prioritario Tarea con prioridad alta labels Feb 29, 2024

negro89 added the En pausa Temporalmente pausado label Mar 7, 2024

github-actions bot added the has_conflicts label May 8, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CIT - Nuevo submódulo Auditoría de Agendas #2882

CIT - Nuevo submódulo Auditoría de Agendas #2882

JuanIRamirez commented Jun 22, 2023 •

edited by negro89

AgosLizzi left a comment •

edited

MCele left a comment •

edited

silviroa commented Jun 28, 2023

silviroa commented Jun 28, 2023

MCele Jun 29, 2023 •

edited

JuanIRamirez Jul 6, 2023

MCele commented Jun 29, 2023

silviroa commented Jun 30, 2023

silviroa commented Jun 30, 2023

silviroa commented Jan 17, 2024

silviroa commented Jan 25, 2024

silviroa commented Jan 25, 2024

silviroa commented Jan 25, 2024

silviroa commented Jan 25, 2024

negro89 commented Feb 29, 2024

CIT - Nuevo submódulo Auditoría de Agendas #2882

Are you sure you want to change the base?

CIT - Nuevo submódulo Auditoría de Agendas #2882

Conversation

JuanIRamirez commented Jun 22, 2023 • edited by negro89

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

AgosLizzi left a comment • edited

Choose a reason for hiding this comment

MCele left a comment • edited

Choose a reason for hiding this comment

silviroa commented Jun 28, 2023

silviroa commented Jun 28, 2023

MCele Jun 29, 2023 • edited

Choose a reason for hiding this comment

JuanIRamirez Jul 6, 2023

Choose a reason for hiding this comment

MCele commented Jun 29, 2023

silviroa commented Jun 30, 2023

silviroa commented Jun 30, 2023

silviroa commented Jan 17, 2024

silviroa commented Jan 25, 2024

silviroa commented Jan 25, 2024

silviroa commented Jan 25, 2024

silviroa commented Jan 25, 2024

negro89 commented Feb 29, 2024

JuanIRamirez commented Jun 22, 2023 •

edited by negro89

AgosLizzi left a comment •

edited

MCele left a comment •

edited

MCele Jun 29, 2023 •

edited