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.

Sign up for GitHub

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

Jump to bottom

INTERNACION - Agregar o sacar columnas adicionales de la tabla #1949

Merged
merged 4 commits into from
Oct 19, 2020
Merged

INTERNACION - Agregar o sacar columnas adicionales de la tabla #1949

merged 4 commits into from
Oct 19, 2020

Conversation

GaboCancellieri
Copy link
Contributor

@GaboCancellieri GaboCancellieri commented Oct 9, 2020 •
edited
Loading

Requerimiento

A demás de las columnas que se muestran ahora(fijas), poder elegir otras columnas para visualizar. Columnas extras: fecha ultimo movimiento, usuario movimiento, sector completo (arrancar de arriba a abajo así se puede sortear), Documento, Sexo

Funcionalidad desarrollada

  1. Agrega columnas adicionales a la tabla
  2. Permite ordenar la tabla por estas columnas también

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

@GaboCancellieri GaboCancellieri requested review from a team as code owners October 9, 2020 17:40
@liquid36
Copy link
Contributor

liquid36 commented Oct 9, 2020

USUARIO: mbotta
BUILD NUMBER: 2163
TEST START: 2020-10-09T19:14:19.709Z
TOTAL: 5
SUCCESS: 5
FAIL: 0
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@liquid36 liquid36 added the changes requested Se solicitaron cambios label Oct 14, 2020
JSantarelli
JSantarelli suggested changes Oct 14, 2020
View reviewed changes
Copy link
Contributor

@JSantarelli JSantarelli left a comment •
edited
Loading

Choose a reason for hiding this comment

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

La funcionalidad en sí es un gran aporte. Sólo un cambio (en la medida en que se pueda implementar) y una observación:

a. Cambio: Cuando el usuario navega hacia otra sección (existe un ruteo) los filtros seleccionados se resetean. Imagino que no es un comportamiento deseado (adjunto gif).

mapa-camas

b. Mejora: En casos críticos (todas las columnas al asador) y una pantalla de 1280 x 768 px se da superposición de elementos (adjunto captura). Quizás logrando que se apilen los badge no sea tan conflictivo.

caso-extremo-2

@liquid36
Copy link
Contributor

a. A futuro despues de detectar el uso que se lede.
b. Por algo la funcionalidad de elegir columna, muchas veces el usuario no necesita ver todo y se le termina rompiendo el layout por exceso de información. que se maneje y seleccione lo que necesite. Esta en camino de plex-list pero no quedan algunos detalles funcionales.
c. Surgio un tema con el cdk-virtual-scroll, y se solucionará con la implementación de plex-list.

@JSantarelli JSantarelli self-requested a review October 15, 2020 11:44
@liquid36 liquid36 added Aprobado and removed changes requested Se solicitaron cambios labels Oct 16, 2020
@liquid36
Copy link
Contributor

USUARIO: mbotta
BUILD NUMBER: 2220
CYPRESS RUN: 1402
TEST START: 2020-10-16T11:07:30.045Z
TOTAL: 322
SUCCESS: 322
FAIL: 0
SKIPPED: 0

@liquid36 liquid36 added the test ok Los test estan ok label Oct 16, 2020
@liquid36 liquid36 merged commit a0b9dce into master Oct 19, 2020
@liquid36 liquid36 deleted the IN-209 branch October 19, 2020 16:44
liquid36 pushed a commit that referenced this pull request Oct 21, 2020
@semantic-release-bot
chore(release): 5.18.0 [skip ci]
7f7debd 
# [5.18.0](v5.17.0...v5.18.0) (2020-10-21)

### Bug Fixes

* **com:** limpiar sidebars al cambiar de derivacion ([#1953](#1953)) ([4f0da19](4f0da19))
* **com:** permitir al com cancelar sus derivaciones solicitadas ([#1955](#1955)) ([4f750f8](4f750f8))
* **GDU:** ExpressionChangedAfterItHasBeenCheckedError perfiles ([#1948](#1948)) ([62c2b06](62c2b06))
* **mapa-camas:** handle null al borrar sector ([#1974](#1974)) ([2390ee8](2390ee8))
* **rup:** solicitudes pierde el formato ([#1957](#1957)) ([f7129e3](f7129e3))

### Features

* **CITAS:** Habilita turnos de gestion en agendas del dia ([#1932](#1932)) ([4c6dc5a](4c6dc5a))
* **com:** marcar derivación seleccionada en plex-list ([#1975](#1975)) ([981daa8](981daa8))
* **com:** se suma fecha de creación/actualización a listado derivaciones ([#1959](#1959)) ([1af2284](1af2284))
* **mapa-camas:** agrega sector completo al abm de sala común ([#1963](#1963)) ([43f3e07](43f3e07))
* **mapa-camas:** columnas dinamicas ([#1949](#1949)) ([a0b9dce](a0b9dce))
* **mapa-camas:** componente de inconsistencias de camas ([#1746](#1746)) ([a2232e3](a2232e3))
* **mapa-camas:** control historial de cama al egresar ([#1880](#1880)) ([7b6c74e](7b6c74e))
* **mapa-camas:** deshacer internación  ([#1945](#1945)) ([67ea6a4](67ea6a4))

### Performance Improvements

* **huds:** quita sort inecesario ([#1952](#1952)) ([55ae989](55ae989))
* **top:** evita calcular estado todo el tiempo ([#1962](#1962)) ([3b358ae](3b358ae))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liquid36 liquid36 liquid36 approved these changes

@JSantarelli JSantarelli Awaiting requested review from JSantarelli

Assignees
No one assigned
Labels
Aprobado test ok Los test estan ok
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@GaboCancellieri @liquid36 @JSantarelli