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.

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

feat(rup): ajuste visual botones en ejecucion #2151

Merged
merged 2 commits into from
Mar 9, 2021
Merged

feat(rup): ajuste visual botones en ejecucion #2151

merged 2 commits into from
Mar 9, 2021

Conversation

liquid36
Copy link
Contributor

@liquid36 liquid36 commented Mar 4, 2021

Requerimiento

https://proyectos.andes.gob.ar/browse/RUP-160

Funcionalidad desarrollada

  1. Mueve los botones al nivel del plex-tabs ganando más espacio.

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

@liquid36 liquid36 requested review from a team as code owners March 4, 2021 18:24
@liquid36
Copy link
Contributor Author

liquid36 commented Mar 4, 2021

USUARIO: mbotta
BUILD NUMBER: 3488
CYPRESS RUN: 2582
TEST START: 2021-03-04T18:37:29.373Z
TOTAL: 363
SUCCESS: 358
FAIL: 5
SKIPPED: 0

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 4, 2021

USUARIO: mbotta
BUILD NUMBER: 3490
CYPRESS RUN: 2584
TEST START: 2021-03-04T19:00:16.862Z
TOTAL: 363
SUCCESS: 358
FAIL: 5
SKIPPED: 0

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 5, 2021

USUARIO: mbotta
BUILD NUMBER: 3493
CYPRESS RUN: 2587
TEST START: 2021-03-05T10:59:59.160Z
TOTAL: 363
SUCCESS: 360
FAIL: 3
SKIPPED: 0

Comment on lines +19 to +21
icon="{{ collapse ? 'chevron-double-down' : 'chevron-double-up'}}"
class="collapse-card"
title="{{ collapse ? 'Expandir los registros' : 'Colapsar los registros'}}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Me parece que debería detectarse el estado inicial, ya que al iniciar una prestación los registros aparecen expandidos (botón: contraer), pero al continuar un ejecución guardada, aparecen contraídos (botón: expandir).

Copy link
Contributor

Choose a reason for hiding this comment

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

Me encantó lo de ngProjectAs="plex-button"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Recién estuve viendo ese comporatamiento, creo que antes deberíamos definir si aparecen colapsados o no. Depende de los valores de los registros y si son requeridos o no.

Deberiamos unificar los criterios.

Copy link
Contributor

Choose a reason for hiding this comment

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

Bien, para mí deberían estar siempre expandidos. Qué decís @JSantarelli?

Copy link
Contributor

Choose a reason for hiding this comment

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

Siempre expandidos y veremos que hace el usuario con Hotjar.

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 5, 2021

USUARIO: mbotta
BUILD NUMBER: 3507
CYPRESS RUN: 2600
TEST START: 2021-03-05T18:03:42.721Z
TOTAL: 313
SUCCESS: 310
FAIL: 3
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 5, 2021

USUARIO: mbotta
BUILD NUMBER: 3510
CYPRESS RUN: 2602
TEST START: 2021-03-05T18:33:41.099Z
TOTAL: 280
SUCCESS: 266
FAIL: 14
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 8, 2021

USUARIO: mbotta
BUILD NUMBER: 3519
CYPRESS RUN: 2609
TEST START: 2021-03-08T11:46:21.897Z
TOTAL: 314
SUCCESS: 300
FAIL: 14
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 8, 2021

USUARIO: mbotta
BUILD NUMBER: 3520
CYPRESS RUN: 2610
TEST START: 2021-03-08T12:00:08.794Z
TOTAL: 363
SUCCESS: 354
FAIL: 9
SKIPPED: 0

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 8, 2021

USUARIO: mbotta
BUILD NUMBER: 3522
CYPRESS RUN: 2612
TEST START: 2021-03-08T12:32:18.644Z
TOTAL: 363
SUCCESS: 363
FAIL: 0
SKIPPED: 0

@liquid36 liquid36 added test ok Los test estan ok and removed test fail labels Mar 8, 2021
Copy link
Contributor

@juuliotero juuliotero left a comment

Choose a reason for hiding this comment

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

ver lo del estado inicial que dicen @andrrrl y @JSantarelli, después funciona está todo ok

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 8, 2021

USUARIO: mbotta
BUILD NUMBER: 3527
CYPRESS RUN: 2617
TEST START: 2021-03-08T15:38:28.557Z
TOTAL: 359
SUCCESS: 357
FAIL: 2
SKIPPED: 0

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 8, 2021

USUARIO: mbotta
BUILD NUMBER: 3528
CYPRESS RUN: 2618
TEST START: 2021-03-08T15:47:32.520Z
TOTAL: 363
SUCCESS: 362
FAIL: 1
SKIPPED: 0

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 8, 2021

USUARIO: mbotta
BUILD NUMBER: 3529
CYPRESS RUN: 2619
TEST START: 2021-03-08T16:10:12.402Z
TOTAL: 355
SUCCESS: 348
FAIL: 7
SKIPPED: 0

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 8, 2021

USUARIO: mbotta
BUILD NUMBER: 3530
CYPRESS RUN: 2620
TEST START: 2021-03-08T16:28:26.573Z
TOTAL: 304
SUCCESS: 292
FAIL: 12
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@palita1991 palita1991 added the En revisión Se está probando label Mar 8, 2021
Copy link
Contributor

@palita1991 palita1991 left a comment

Choose a reason for hiding this comment

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

Revisar que se subio el archivo src/environments/environment.dev.ts apuntando la api a demo, lo demás todo ok 👍

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 8, 2021

USUARIO: mbotta
BUILD NUMBER: 3531
CYPRESS RUN: 2621
TEST START: 2021-03-08T16:50:16.574Z
TOTAL: 363
SUCCESS: 362
FAIL: 1
SKIPPED: 0

@liquid36
Copy link
Contributor Author

liquid36 commented Mar 8, 2021

USUARIO: mbotta
BUILD NUMBER: 3532
CYPRESS RUN: 2622
TEST START: 2021-03-08T17:03:19.793Z
TOTAL: 363
SUCCESS: 363
FAIL: 0
SKIPPED: 0

@liquid36 liquid36 added test ok Los test estan ok and removed test fail labels Mar 8, 2021
@liquid36 liquid36 merged commit fe4ef33 into master Mar 9, 2021
@liquid36 liquid36 deleted the RUP-160 branch March 9, 2021 16:39
liquid36 pushed a commit that referenced this pull request Mar 10, 2021
# [5.37.0](v5.36.0...v5.37.0) (2021-03-10)

### Bug Fixes

* **discapacidad:** quita restricción a grupo discapacidad para limite de edad maxima ([01c562e](01c562e))
* **epidemiologia:** boton general para registrar ficha ([b8cbb4f](b8cbb4f))
* **epidemiologia:** change fichas, virtualScroll ([caba926](caba926))
* **ficha-epidemiologica:** agregamos información que falta en el test ([#2154](#2154)) ([a76dda1](a76dda1))
* **form-crud:** modificamos campos de acuerdo a nueva actualizacion de api ([2741d5a](2741d5a))
* **forms-crud:** altura extra para plex-select ([6387bf3](6387bf3))
* **mapa-camas:** movimientos en listado internacion ([#2159](#2159)) ([33c3363](33c3363))
* **RUP:** Se agrega un control en vacunas para saber si se debe hacer el chequeo interdosis o no ([#2160](#2160)) ([0f9eb70](0f9eb70))

### Features

* **epidemiologia:** agrego component seccion ([b211b69](b211b69))
* **epidemiologia:** nueva seccion ([f0d8219](f0d8219))
* **epidemiologia:** nueva tabla contactosEstrechos ([f08fe87](f08fe87))
* **epidemiologia:** nuevo modulo y ficha-epidemiologica component ([c509bba](c509bba))
* **epidemiologia:** save y update ficha ([02f02a4](02f02a4))
* **epidemiologia:** table contactos estrechos ([c84018b](c84018b))
* **epidemiologia:** table contactos estrechos ([b4b7998](b4b7998))
* **ficha-epidemio:** altura formBuilder, ficha-crud, permisos ([c4365da](c4365da))
* **formBuilder:** Add snomed codification to fields and form ([9e6f995](9e6f995))
* **formBuilder:** add type table ([1fba268](1fba268))
* **formBuilder:** agregamos plex-table y fixeamos algunos detalles ([5405a17](5405a17))
* **formBuilder:** formulario generico se le agregan secciones por campo ([1cffe2f](1cffe2f))
* **formBuilder:** primer commit ([cf21663](cf21663))
* **formBuilder:** set subfilter false by default ([0dde344](0dde344))
* **forms:** Add phone option ([12a7a27](12a7a27))
* **mpi:** reemplaza toast de ciudadano no encontrado ([#2146](#2146)) ([491715b](491715b))
* **rup:** ajuste visual botones en ejecucion ([#2151](#2151)) ([fe4ef33](fe4ef33))

### Performance Improvements

* **turnos-prestaciones:** maximo de prestaciones para exportar ([#2150](#2150)) ([4725e23](4725e23))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants