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

CIT - Verifica turnos y solicitudes al asignar demanda #3091

Merged
merged 1 commit into from
Dec 18, 2024
Merged

Conversation

ma7payne
Copy link
Contributor

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-344

Funcionalidad desarrollada

  1. Pide datos de solicitudes pendientes asociadas al paciente y la prestación
  2. Si existen solicitudes se muestra modal informando de las mismas
  3. Finaliza cierre de demanda al cerrar modal

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-345

Funcionalidad desarrollada

  1. Pide datos de turnos futuros asociados al paciente y la prestación
  2. Si existe turno se evita el guardado de la demanda y se muestra modal notificando los datos del turno

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

@MCele MCele self-requested a review November 21, 2024 15:10
@@ -22,7 +22,7 @@ export class ListaEsperaService {
return this.server.get(this.listaEsperaUrl, { params: params, showError: true });
}

post(listaEspera: IListaEspera): Observable<IListaEspera> {
post(listaEspera: IListaEspera): Observable<any> {
Copy link
Contributor

Choose a reason for hiding this comment

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

Mati estuvimos revisando y los modal se ven y funcionan perfecto, pero al momento de cerrar una demanda falla el request a la api
image

Por otro lado, por lo que estuvimos viendo y analizando es que el post que se modifica en la api da respuestas distintas dependiendo del caso y por ahi nos queda dudas si eso es apropiado, es decir:

  1. si existe un turno, esta bien que no lo guarde, pero la respuesta quizás no deba ser status 200, porque no fue exitosa la operación.
  2. si no existen turnos, y si existen solicitudes, eso quizás debería ser un control externo al post, ya que no modifican su comportamiento. Lo que vimos también en ese caso, es que la app en ventanilla-citas, ya resuelve ese mismo request de solicitudes, quizás se pueda desde ahí reutilizar ese servicio. De no ser así, habría que evaluar si es necesario en api generar una consulta nueva y no utilizar la que ya tiene prestaciones, que es dónde debería estar la consulta a la BD.

Mati cualquier duda consultanos a mi o a Lauchita y lo charlamos

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@MCele y @negro89 estan realizados los siguientes cambios:

  • Llama al servicio correspondiente para guardar la demanda.
  • Si existe turno API devuelve un error 500 y muestra un modal con los detalles del turno. Se interrumpe el flujo para que no se recuperen las solicitudes pendientes.
  • Si el guardado es exitoso, consulta las solicitudes relacionadas desde APP, invocando PrestacionesService. Si encuentra solicitudes asociadas, las muestra en un modal. Se utiliza switchMap para evitar subscribes anidados.
  • Si no hay solicitudes, muestra un mensaje confirmando que la demanda fue guardada con éxito y cierra el flujo.
  • Se agrega la opción { showError: false } en listaEspera.service para evitar mostrar el modal de error por defecto.

Quedo atento a las correcciones!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Con respecto al error de guardado de demandas, queda solucionado con el fix en #3113.

@ma7payne ma7payne added the changes requested Se solicitaron cambios label Nov 25, 2024
@ma7payne ma7payne force-pushed the CIT-345 branch 4 times, most recently from fa326b4 to 069b226 Compare November 27, 2024 16:06
@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels Nov 27, 2024
@ma7payne ma7payne force-pushed the CIT-345 branch 4 times, most recently from 9a03ead to e4535e8 Compare November 27, 2024 17:16
@MCele MCele self-requested a review December 16, 2024 12:10
@negro89 negro89 merged commit b92f319 into master Dec 18, 2024
2 checks passed
@negro89 negro89 deleted the CIT-345 branch December 18, 2024 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants