CIT: Implementa mockup para sobre-turnos en ventanilla de Citas #2826

ma7payne · 2023-03-08T18:12:58Z

Requerimiento

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

Funcionalidad desarrollada

En el sidebar abierto tras presionar "Dar Turno" dentro del Calendario de Agenda, se visualiza el botón "Agregar Sobreturno".
Al activar la opción para agregar sobreturno se muestra el formulario de carga con los datos de la agenda ligados al paciente seleccionado.
Se permite la carga, validación y guardado del sobreturno sin cambiar de instancia.
Al finalizar el guardado del sobreturno, se obtienen los datos del mismo para mostrarlos en un modal.
Al cerrar el modal, se redirecciona a la pantalla inicial de ventanilla de Citas, con el paciente actual.

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

silviroa · 2023-03-08T18:44:28Z

USUARIO: mchiappe
BUILD NUMBER: 8559
CYPRESS RUN: 7737
TEST START: 2023-03-08T18:44:09.763Z
TOTAL: 406
SUCCESS: 403
FAIL: 1
SKIPPED: 2

silviroa · 2023-03-09T13:51:47Z

USUARIO: mchiappe
BUILD NUMBER: 8561
CYPRESS RUN: 7739
TEST START: 2023-03-09T13:51:28.133Z
TOTAL: 406
SUCCESS: 404
FAIL: 0
SKIPPED: 2

MarianoCampetella · 2023-03-15T14:27:35Z

src/app/components/turnos/dar-turnos/dar-sobreturno.component.ts

+            this._agenda = agenda;
+            this.agenda = agenda;
+            this.inicio = new Date(this.hoy.setHours(this.agenda.horaInicio.getHours(), this.agenda.horaInicio.getMinutes(), 0, 0));
+            this.fin = new Date(this.hoy.setHours(this.agenda.horaFin.getHours(), this.agenda.horaFin.getMinutes(), 0, 0));


Actualmente se esta utilizando moment() para trabajar con todo tipo de fechas, incluso para obtener hora, minutos, segundos, etc.

Listo los cambios!

MarianoCampetella · 2023-03-15T14:29:37Z

src/app/components/turnos/dar-turnos/dar-sobreturno.component.ts

+    // a partir del documento y del efector
+    obtenerCarpetaPaciente() {
+        let indiceCarpeta = -1;
+        if (this.paciente.carpetaEfectores.length > 0) {


No es necesario tener que aclarar en un .length que tiene que ser mayer a cero.
Con hacer if (this.paciente.carpetaEfectores.length) ya se esta evaluando si es mayor a cero.

Cambios realizados!

MarianoCampetella · 2023-03-15T14:31:36Z

También es necesario eliminar las variables que no fueron utilizadas tanto en el html como en el .ts
loading, resultadoBusqueda, resultado, pacientes, seleccion, esEscaneado, etc.

AgosLizzi

Mati, un mini detalle:
Generar una consistencia con los colores de los datos del sobreturno, tenemos 2 datos en celeste, 1 en verde y 1 en negro. Si no tienen alguna explicación logica, los pasaria a todos al mismo color. Tal vez me quedaría con el negro, para cortar un poco con los colores

ma7payne · 2023-03-28T13:19:15Z

Han sido realizados los cambios de fecha, variables sin usar y length inapropiados. También los colores en el modal. Quedo atento a las correcciones @AgosLizzi @MarianoCampetella .

MarianoCampetella · 2023-03-16T11:58:15Z

src/app/components/turnos/dar-turnos/dar-sobreturno.component.ts

+    }
+
+    actualizarCarpetaPaciente() {
+        this.servicePaciente.patch(this.paciente.id, { op: 'updateCarpetaEfectores', carpetaEfectores: this.paciente.carpetaEfectores }).subscribe(resultadoCarpeta => {


Si dentro de un subscribe no se utiliza la variable que se define, por ejemplo resultadoCarpeta, lo mejor es dejarlo con parentesis vacio. Ejemplo:
... .subscribe( () => {
...
})

negro89 · 2023-04-11T14:33:59Z

src/app/components/turnos/dar-turnos/dar-sobreturno.component.ts

+        if (paciente.contacto) {
+            if (paciente.contacto.length) {


Suggested change

if (paciente.contacto) {

if (paciente.contacto.length) {

if (paciente.contacto?.length) {

o incluso para la linea de abajo, sin el if anterior, tambien sería valido

Suggested change

if (paciente.contacto) {

if (paciente.contacto.length) {

paciente.contacto?.forEach((contacto) => {

negro89 · 2023-04-11T15:07:26Z

Mati, el nombre y apellio del paciente es redundante ya que aparecen en el apartado de arriba

MCele

Veo que al momento de cambiar de agendas el rango de horarios posibles para los sobre-turnos no se actualiza a la agenda. Luego de esto el paciente queda cargado en la agenda original y no en la que se está viendo en el momento.
Ejemplo:
Selecciono agenda 1: fecha Viernes 21/4/2023 13:00 - 17:00hs. Click en sobreturno (se carga el componente).
Luego selecciono la agenda 2: Lunes 24/4/2023 08:00 - 11:00hs, se visualiza la agenda ultima elegida, pero el rango de horarios es de la agenda 1

Cargo sobreturno para la agenda 2:

Finalmente se almacena el sobre-turno en la agenda 1:

silviroa · 2023-04-27T14:12:08Z

USUARIO: mchiappe
BUILD NUMBER: 8783
CYPRESS RUN: 7954
TEST START: 2023-04-27T14:10:54.122Z
TOTAL: 336
SUCCESS: 325
FAIL: 9
SKIPPED: 2

silviroa · 2023-04-27T17:11:03Z

USUARIO: cramos
BUILD NUMBER: 8790
CYPRESS RUN: 7960
TEST START: 2023-04-27T17:10:48.846Z
TOTAL: 406
SUCCESS: 404
FAIL: 0
SKIPPED: 2

MCele · 2023-04-27T19:45:39Z

src/app/components/turnos/dar-turnos/dar-sobreturno.component.ts

+    public lenNota = 140;
+    public changeCarpeta: boolean;
+    public carpetaEfector: any;
+    public _agenda;


cuidado con copiar y pegar... revisar todas las variables utilizadas, ya que si son innecesarias no deberían "arrastrarse" a este componente

silviroa · 2023-04-28T15:31:08Z

USUARIO: mchiappe
BUILD NUMBER: 8797
CYPRESS RUN: 7967
TEST START: 2023-04-28T15:29:55.678Z
TOTAL: 369
SUCCESS: 358
FAIL: 9
SKIPPED: 2

silviroa · 2023-04-28T16:11:45Z

USUARIO: mchiappe
BUILD NUMBER: 8798
CYPRESS RUN: 7968
TEST START: 2023-04-28T16:11:31.594Z
TOTAL: 406
SUCCESS: 403
FAIL: 1
SKIPPED: 2

silviroa · 2023-04-28T16:31:45Z

USUARIO: mchiappe
BUILD NUMBER: 8799
CYPRESS RUN: 7969
TEST START: 2023-04-28T16:31:32.714Z
TOTAL: 388
SUCCESS: 385
FAIL: 1
SKIPPED: 2

silviroa · 2023-04-28T16:45:32Z

USUARIO: mchiappe
BUILD NUMBER: 8800
CYPRESS RUN: 7970
TEST START: 2023-04-28T16:45:16.580Z
TOTAL: 406
SUCCESS: 404
FAIL: 0
SKIPPED: 2

# [5.117.0](v5.116.0...v5.117.0) (2023-05-03) ### Bug Fixes * **IN-519:** Solapamiento de funcionalidades en el sidebar ([#2835](#2835)) ([b8362d0](b8362d0)) * **IN-526:** Fijar botones al cargar una indicación ([#2850](#2850)) ([48a7ceb](48a7ceb)) ### Features * **CIT:** implementa mockup en sobreturno ([#2826](#2826)) ([7ca0702](7ca0702)) * **RUP:** no mostrar la prestacion que se utiliza ([#2721](#2721)) ([bc0b9a2](bc0b9a2))

ma7payne requested review from a team as code owners March 8, 2023 18:12

ma7payne requested review from MCele and MarianoCampetella March 8, 2023 18:12

silviroa added the test fail label Mar 8, 2023

ma7payne force-pushed the CIT-209 branch from aff1bfd to c815f9f Compare March 9, 2023 13:26

silviroa added test ok Los test estan ok and removed test fail labels Mar 9, 2023

MarianoCampetella requested changes Mar 15, 2023

View reviewed changes

MarianoCampetella added the changes requested Se solicitaron cambios label Mar 16, 2023

AgosLizzi requested changes Mar 20, 2023

View reviewed changes

ma7payne force-pushed the CIT-209 branch from c815f9f to fdf3871 Compare March 28, 2023 13:14

ma7payne added changes done and removed changes requested Se solicitaron cambios labels Mar 28, 2023

AgosLizzi self-requested a review March 29, 2023 12:26

AgosLizzi approved these changes Mar 29, 2023

View reviewed changes

MarianoCampetella approved these changes Mar 29, 2023

View reviewed changes

negro89 reviewed Apr 11, 2023

View reviewed changes

negro89 force-pushed the CIT-209 branch from fdf3871 to cd9354c Compare April 11, 2023 14:49

negro89 added changes requested Se solicitaron cambios and removed changes done labels Apr 11, 2023

MCele requested changes Apr 21, 2023

View reviewed changes

MCele self-assigned this Apr 27, 2023

MCele added the publicar label Apr 27, 2023

silviroa added test fail and removed test ok Los test estan ok labels Apr 27, 2023

MCele force-pushed the CIT-209 branch from e491462 to a538744 Compare April 27, 2023 16:56

silviroa added test ok Los test estan ok and removed test fail labels Apr 27, 2023

MCele reviewed Apr 27, 2023

View reviewed changes

feat(CIT): implementa mockup en sobreturno

9cb2b85

ma7payne force-pushed the CIT-209 branch from a538744 to 9cb2b85 Compare April 28, 2023 14:38

silviroa added test fail and removed test ok Los test estan ok labels Apr 28, 2023

MCele self-requested a review April 28, 2023 15:58

MCele approved these changes Apr 28, 2023

View reviewed changes

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

silviroa added test ok Los test estan ok and removed test fail labels Apr 28, 2023

negro89 approved these changes May 2, 2023

View reviewed changes

negro89 added Aprobado and removed changes done labels May 2, 2023

negro89 merged commit 7ca0702 into master May 2, 2023

negro89 deleted the CIT-209 branch May 2, 2023 15:02

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CIT: Implementa mockup para sobre-turnos en ventanilla de Citas #2826

CIT: Implementa mockup para sobre-turnos en ventanilla de Citas #2826

ma7payne commented Mar 8, 2023 •

edited

Loading

silviroa commented Mar 8, 2023

silviroa commented Mar 9, 2023

MarianoCampetella Mar 15, 2023

ma7payne Mar 28, 2023

MarianoCampetella Mar 15, 2023

ma7payne Mar 28, 2023

MarianoCampetella commented Mar 15, 2023

AgosLizzi left a comment •

edited

Loading

ma7payne commented Mar 28, 2023

MarianoCampetella Mar 16, 2023

negro89 Apr 11, 2023

negro89 commented Apr 11, 2023

MCele left a comment •

edited

Loading

silviroa commented Apr 27, 2023

silviroa commented Apr 27, 2023

MCele Apr 27, 2023 •

edited

Loading

silviroa commented Apr 28, 2023

silviroa commented Apr 28, 2023

silviroa commented Apr 28, 2023

silviroa commented Apr 28, 2023

	if (paciente.contacto) {
	if (paciente.contacto.length) {
	if (paciente.contacto?.length) {

	if (paciente.contacto) {
	if (paciente.contacto.length) {
	paciente.contacto?.forEach((contacto) => {

CIT: Implementa mockup para sobre-turnos en ventanilla de Citas #2826

CIT: Implementa mockup para sobre-turnos en ventanilla de Citas #2826

Conversation

ma7payne commented Mar 8, 2023 • 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

silviroa commented Mar 8, 2023

silviroa commented Mar 9, 2023

MarianoCampetella Mar 15, 2023

Choose a reason for hiding this comment

ma7payne Mar 28, 2023

Choose a reason for hiding this comment

MarianoCampetella Mar 15, 2023

Choose a reason for hiding this comment

ma7payne Mar 28, 2023

Choose a reason for hiding this comment

MarianoCampetella commented Mar 15, 2023

AgosLizzi left a comment • edited Loading

Choose a reason for hiding this comment

ma7payne commented Mar 28, 2023

MarianoCampetella Mar 16, 2023

Choose a reason for hiding this comment

negro89 Apr 11, 2023

Choose a reason for hiding this comment

negro89 commented Apr 11, 2023

MCele left a comment • edited Loading

Choose a reason for hiding this comment

silviroa commented Apr 27, 2023

silviroa commented Apr 27, 2023

MCele Apr 27, 2023 • edited Loading

Choose a reason for hiding this comment

silviroa commented Apr 28, 2023

silviroa commented Apr 28, 2023

silviroa commented Apr 28, 2023

silviroa commented Apr 28, 2023

ma7payne commented Mar 8, 2023 •

edited

Loading

AgosLizzi left a comment •

edited

Loading

MCele left a comment •

edited

Loading

MCele Apr 27, 2023 •

edited

Loading