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

feat(GDU): Control profesion sin numero de matricula #2496

Merged
merged 1 commit into from
Oct 21, 2021
Merged

feat(GDU): Control profesion sin numero de matricula #2496

merged 1 commit into from
Oct 21, 2021

Conversation

palita1991
Copy link
Contributor

@palita1991 palita1991 commented Oct 13, 2021 •
edited by kroce
Loading

Requerimiento

https://proyectos.andes.gob.ar/browse/GDU-38

Funcionalidad desarrollada

  1. Cuando el profesional está matriculado, pero no tiene numero de matricula se rompia el gestor de usuarios por los nuevos badges, por lo que se agregó un control para esos casos

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

@palita1991 palita1991 requested a review from a team as a code owner October 13, 2021 12:54
@palita1991 palita1991 self-assigned this Oct 13, 2021
@palita1991 palita1991 requested review from a team as code owners October 13, 2021 12:54
@liquid36
Copy link
Contributor

USUARIO: mpalavecino
BUILD NUMBER: 5284
CYPRESS RUN: 4229
TEST START: 2021-10-13T13:22:40.484Z
TOTAL: 401
SUCCESS: 398
FAIL: 2
SKIPPED: 1

@liquid36
Copy link
Contributor

USUARIO: mpalavecino
BUILD NUMBER: 5288
CYPRESS RUN: 4233
TEST START: 2021-10-13T16:12:15.783Z
TOTAL: 401
SUCCESS: 400
FAIL: 0
SKIPPED: 1

@liquid36 liquid36 added test ok Los test estan ok and removed test fail labels Oct 13, 2021
juuliotero
juuliotero suggested changes Oct 13, 2021
View reviewed changes
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.

Cuando borro el array de matriculación de formación de grado del profesional, se rompe el badge y hace algo raro con la matricula de otro profesional

https://www.loom.com/share/568625f2e052438cb2f496cf039bb4b6

@palita1991
Copy link
Contributor Author

Cuando borro el array de matriculación de formación de grado del profesional, se rompe el badge y hace algo raro con la matricula de otro profesional

https://www.loom.com/share/568625f2e052438cb2f496cf039bb4b6

@juuliotero corregido

@juuliotero
Copy link
Contributor

Cuando borro el array de matriculación de formación de grado del profesional, se rompe el badge y hace algo raro con la matricula de otro profesional
https://www.loom.com/share/568625f2e052438cb2f496cf039bb4b6

@juuliotero corregido

Si no tiene número podría ocultar el 'N°'
image

@palita1991
Copy link
Contributor Author

Cuando borro el array de matriculación de formación de grado del profesional, se rompe el badge y hace algo raro con la matricula de otro profesional
https://www.loom.com/share/568625f2e052438cb2f496cf039bb4b6

@juuliotero corregido

Si no tiene número podría ocultar el 'N°' image

@juuliotero Corregido

plammel
plammel reviewed Oct 13, 2021
View reviewed changes
src/app/apps/gestor-usuarios/components/usuario-detalle/usuario-detalle.html
Comment on lines +8 to +11
profesional.formacionGrado[0].profesion.nombre +' '+(profesional.formacionGrado[0].matriculacion?.length?
'MATRÍCULA N° '
+profesional.formacionGrado[0].matriculacion[profesional.formacionGrado[0].matriculacion?.length
-1].matriculaNumero:''):'NO MATRICULADO'}}</span>
Copy link
Contributor

Choose a reason for hiding this comment

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

@palita1991 Palita, al leer se hace bastante difícil entender lo que esta sucediendo (dos asignaciones ternarias anidadas). Tal vez un mejor identado y uso de espacios ayude a facilitar la legibilidad.
Por otro lado, se va a mostrar MATRICULA N° ## si tiene matricula c/ número, y 'NO MATRICULADO', si no tiene matricula, pero si tiene matricula y está sin número no va a mostrar ninguna información, es el comportamiento esperado?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@plammel rulito, el tema del identado lo hace automático la configuración que tenemos, no se si puedo manejar mucho ese tema a mano. En cuanto a la información que se va a mostrar, estas en lo correcto.

@kroce kroce merged commit 4013e33 into master Oct 21, 2021
@kroce kroce deleted the GDU-38 branch October 21, 2021 17:28
liquid36 pushed a commit that referenced this pull request Oct 27, 2021
@semantic-release-bot
chore(release): 5.70.0 [skip ci]
88dd13d 
# [5.70.0](v5.69.0...v5.70.0) (2021-10-27)

### Bug Fixes

* **novedades:** se alinean títulos de items ([#2501](#2501)) ([f8f7127](f8f7127))

### Features

* **bi:** filtros por organización y zona en dashboard de citas ([#2502](#2502)) ([c2d6d3e](c2d6d3e))
* **ficha:** caso asintomatico ([#2482](#2482)) ([710f636](710f636))
* **GDU:** control para cuando el profesional no tiene numero de matricula ([#2496](#2496)) ([4013e33](4013e33))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@plammel plammel plammel approved these changes

@MCele MCele MCele approved these changes

@juuliotero juuliotero juuliotero approved these changes

@condorpiedra condorpiedra Awaiting requested review from condorpiedra condorpiedra is a code owner automatically assigned from andes/code-review

Assignees

@palita1991 palita1991

Labels
test ok Los test estan ok
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@palita1991 @liquid36 @juuliotero @MCele @plammel @kroce