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

HUDS: fix ordenar prestaciones #2361

Merged
merged 1 commit into from
Jul 14, 2021
Merged

HUDS: fix ordenar prestaciones #2361

merged 1 commit into from
Jul 14, 2021

Conversation

plammel
Copy link
Contributor

@plammel plammel commented Jul 7, 2021

Requerimiento

Condición para ordenar por fecha de creacion en caso de ficha de epidemilogia

Funcionalidad desarrollada

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

@plammel plammel requested review from a team as code owners July 7, 2021 17:19
@@ -63,8 +63,8 @@ export class HudsBusquedaComponent implements AfterContentInit {
}
set prestaciones(value) {
this._prestaciones = value.sort((a, b) => {
const dateA = a.data.ejecucion ? a.data.ejecucion.fecha : a.data.fecha;
const dateB = b.data.ejecucion ? b.data.ejecucion.fecha : b.data.fecha;
const dateA = a.data.ejecucion ? a.data.ejecucion.fecha : (a.data.fecha ? a.data.fecha : a.data.createdAt);
Copy link
Contributor

Choose a reason for hiding this comment

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

en verdad hay que ordenar por a.fecha y b.fecha. Cada tipo de datos setea su fecha antes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updeteado!

Copy link
Contributor

@liquid36 liquid36 left a comment

Choose a reason for hiding this comment

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

Vuelvo a comentar, habría que ordenar por a.fecha y no por a.data.fecha. El tema esta en la función de sort que se accede a data y eso es polimórfico.

@liquid36
Copy link
Contributor

USUARIO: mbotta
BUILD NUMBER: 4568
CYPRESS RUN: 3583
TEST START: 2021-07-14T11:21:19.159Z
TOTAL: 403
SUCCESS: 402
FAIL: 0
SKIPPED: 1

@liquid36 liquid36 added the test ok Los test estan ok label Jul 14, 2021
@liquid36 liquid36 merged commit 3a5d087 into master Jul 14, 2021
@liquid36 liquid36 deleted the fix-ordenamiento-rup branch July 14, 2021 13:47
liquid36 pushed a commit that referenced this pull request Jul 14, 2021
# [5.55.0](v5.54.0...v5.55.0) (2021-07-14)

### Bug Fixes

* **ficha-epidemio:** set mpi ([6e7987e](6e7987e))
* **huds:** fix ordenar prestaciones ([#2361](#2361)) ([3a5d087](3a5d087))
* **HUDS:** Mostrar ficha ([#2360](#2360)) ([e54893c](e54893c))
* **mapa-camas:** filtra camas inactivas ([#2358](#2358)) ([8ddd766](8ddd766))
* **mapa-camas:** hisotrial de un año maxima ([#2365](#2365)) ([06feead](06feead))

### Features

* **EP:** fix duplicidad de save ([#2356](#2356)) ([01c39cb](01c39cb))
* **mapa-camas:** retry operation ([#2362](#2362)) ([37371ed](37371ed))
* **PDP:** creación de componente para actualizar datos personales ([#2328](#2328)) ([01f2b9c](01f2b9c))
* **rup:** romper validacion si tengo permisos ([#2355](#2355)) ([31a85dd](31a85dd))
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.

2 participants