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

RUP - Implementa habilitación de prestaciones a profesional #3014

Merged
merged 1 commit into from
Jul 18, 2024
Merged

RUP - Implementa habilitación de prestaciones a profesional #3014

merged 1 commit into from
Jul 18, 2024

Conversation

ma7payne
Copy link
Contributor

Requerimiento

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

Funcionalidad desarrollada

  1. Crea módulo de habilitación para prestaciones de un profesional
  2. Implementa mockups con formulario de carga de nuevo acceso.

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

@ma7payne ma7payne requested review from a team as code owners May 29, 2024 14:57
@ma7payne ma7payne requested review from MCele and AgosLizzi May 29, 2024 14:57
@ma7payne ma7payne force-pushed the RUP-399 branch 6 times, most recently from cbe2d25 to 668d59a Compare May 30, 2024 14:11
MarianoCampetella
MarianoCampetella requested changes May 30, 2024
View reviewed changes
src/app/app.routing.ts Outdated
@@ -33,6 +33,7 @@ import { PlanificarAgendaComponent } from './components/turnos/gestor-agendas/op
import { RevisionAgendaComponent } from './components/turnos/gestor-agendas/operaciones-agenda/revision-agenda.component';
import { AgregarSobreturnoComponent } from './components/turnos/gestor-agendas/operaciones-agenda/sobreturno.component';
import { PuntoInicioTurnosComponent } from './components/turnos/punto-inicio/puntoInicio-turnos.component';
import { PrestacionesHabilitadasComponent } from './components/turnos/gestor-agendas/prestaciones-habilitadas/prestaciones-habilitadas.component';
// ... MPI
// import { ExtranjeroNNCruComponent } from './core/mpi/components/extranjero-nn-cru.component';
Copy link
Contributor

Choose a reason for hiding this comment

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

Se podría quitar este import comentado.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

✅

...ponents/turnos/gestor-agendas/prestaciones-habilitadas/prestaciones-habilitadas.component.ts Outdated
public diasSemana = [
{ id: 'lunes', label: 'Lunes' },
{ id: 'martes', label: 'Martes' },
{ id: 'miercoles', label: 'Miercoles' },
Copy link
Contributor

Choose a reason for hiding this comment

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

Falta el acento a Miércoles

Copy link
Contributor Author

Choose a reason for hiding this comment

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

✅

@MarianoCampetella
Copy link
Contributor

A modo de comentario, cuando se tenga que continuar con el PR la variable nuevoAcceso habría que cargarle los elementos necesarios para evitar los errores en el HTML de Property ' ... ' does not exist on type '{}'. el resto se encuentra en perfecta condiciones para continuarlo.

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label May 30, 2024
@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels Jun 4, 2024
@ma7payne ma7payne force-pushed the RUP-399 branch 2 times, most recently from a4f09e4 to e440732 Compare June 4, 2024 14:57
@ma7payne
Copy link
Contributor Author

ma7payne commented Jun 4, 2024

A modo de comentario, cuando se tenga que continuar con el PR la variable nuevoAcceso habría que cargarle los elementos necesarios para evitar los errores en el HTML de Property ' ... ' does not exist on type '{}'. el resto se encuentra en perfecta condiciones para continuarlo.

@MarianoCampetella listos los cambios! En relacional ultimo comentario, ya se charló en persona.

negro89
negro89 reviewed Jun 5, 2024
View reviewed changes
Copy link
Contributor

@negro89 negro89 left a comment

Choose a reason for hiding this comment

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

Se ve de 10! solo dos observaciones sobre la pantalla inicial:

  1. El botón "volver" lo venimos ubicando siempre a la izquierda (excepto en los sidebar, que lo mostramos con el icono "X") tal cual la pantalla "crear nuevo acceso"
  2. La leyenda "no hay permisos disponibles" quizás quede mejor dandole un margen en la parte superior para distribuir un poquito mas que nada (solo es una apreciación personal)

image

@MarianoCampetella MarianoCampetella added changes requested Se solicitaron cambios and removed Aprobado labels Jun 6, 2024
@ma7payne
Copy link
Contributor Author

Se ve de 10! solo dos observaciones sobre la pantalla inicial:

  1. El botón "volver" lo venimos ubicando siempre a la izquierda (excepto en los sidebar, que lo mostramos con el icono "X") tal cual la pantalla "crear nuevo acceso"
  2. La leyenda "no hay permisos disponibles" quizás quede mejor dandole un margen en la parte superior para distribuir un poquito mas que nada (solo es una apreciación personal)

image

Se ve de 10! solo dos observaciones sobre la pantalla inicial:

  1. El botón "volver" lo venimos ubicando siempre a la izquierda (excepto en los sidebar, que lo mostramos con el icono "X") tal cual la pantalla "crear nuevo acceso"
  2. La leyenda "no hay permisos disponibles" quizás quede mejor dandole un margen en la parte superior para distribuir un poquito mas que nada (solo es una apreciación personal)

image

@negro89 Cambios listos!

@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels Jun 24, 2024
@ma7payne
Copy link
Contributor Author

TEST OK #11118

@ma7payne ma7payne added the test ok Los test estan ok label Jun 26, 2024
@negro89 negro89 self-requested a review June 28, 2024 14:02
@negro89 negro89 merged commit ba954c2 into master Jul 18, 2024
2 checks passed
@negro89 negro89 deleted the RUP-399 branch July 18, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@negro89 negro89 negro89 approved these changes

@MarianoCampetella MarianoCampetella MarianoCampetella approved these changes

@AgosLizzi AgosLizzi AgosLizzi approved these changes

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

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.

None yet
4 participants
@ma7payne @MarianoCampetella @negro89 @AgosLizzi