udistrital / planeacion_cliente

:heavy_check_mark: Check: Cliente del proyecto de planeación de la universidad Distrital
0 stars 0 forks source link

Refactorización cliente planeacion_seguimiento_mf - PT 2 #1013

Closed milo19980525 closed 5 months ago

milo19980525 commented 5 months ago

Se requiere refactorizar el MF PLANEACION_SEGUIMIENTO_MF con base en el API PLANEACION_CLIENTE, de forma tal que se culmine con la migración de componentes y se presente el cliente en su forma final.

Especificaciones técnicas

  1. El desarrollo de la parte 1 de esta tarea corresponde a lo documentado en el issue #1007 .

  2. El repositorio para PLANEACION_SEGUIMIENTO_MF es: https://github.com/udistrital/planeacion_seguimiento_mf

  3. Se requiere migrar los nuevos desarrollos (en caso de ser requerido) asociados a Reportes, del cliente PLANEACION_CLIENTE al MF PLANEACION_SEGUIMIENTO_MF. Tener en cuenta todos los ajustes, como por ejemplo: #982 , #983 , #997 , #1004 , #1017 entre otras.

  4. El Cliente debe cumplir con los lineamientos establecidos para la creación de nuevos Clientes definidos por la OATI.

  5. Los componentes y servicios que son globales a todos los MF deben ser migrados al cliente CORE o de UTILIDADES.

  6. Se debe optimizar el código migrado, eliminando malas practicas de desarrollo.

  7. El MF debe quedar optimizado en estilos y con pruebas funcionales en su totalidad.

  8. El MF debe consumir los endpoints asociados a los MicroMids desarrollados para SISGPLAN, en lugar de planeacion_mid.

  9. Generar el archivo drone.yml con base en lo encontrado en planeacion_utilidades_mf: pruebaspsisgplanutilidadesmf.portaloas.udistrital.edu.co

  10. De igual manera tener en cuenta los clientes ya desplegados en pruebas:

  11. Actualizar el Readme.md.

Sub Tareas

Criterios de aceptación

Requerimientos

Dependencias

Definition of Ready - DoR

Definition of Done - DoD - Desarrollo

milo19980525 commented 5 months ago

En esta issue no se debe abordar nada relacionado con el ajuste de roles #1017 y #997 . Solo verificar el readme, drone y ajuste de rutas con base en planeacion_seguimiento_mid.

milo19980525 commented 5 months ago

Tener en cuenta este ajuste en el package-json: https://github.com/udistrital/planeacion_utilidades_mf/commit/b2cb27067eaf37ed81aa798cbb323ee14f610c15

C0C045 commented 5 months ago

Refactorización cliente planeacion_seguimiento_mf - PT 2

  1. Se ajustaron todas rutas qué apuntaban a 'PLANES_MID' para que usarán los micromid's de 'FORMULACION' y 'SEGUIMIENTO', además de ajustar la respuestas de los datos, ya que antes se esperaba un 'Data' y ahora 'data'; tal que así: Caso 'FORMULACION_MID': image Caso 'SEGUIMIENTO_MID': image Prueba Funcional: Pruebas Mids Nota: Como podemos ver en la última parte de la prueba funcional cuando queremos ver el detalle de una actividad en especifico, la consulta retorna un 500 indicando que la consulta: 'SEGUIMIENTO_MID/seguimiento/65bf03d30c1fc945b06aee5b/1/635b1fff5073f2675157dc83' esta fallando, por ende, es necesario indagar en el posible error y solucionarlo. image

  2. Por otro lado tanto el drone.yml y el README.md quedaron actualizados según lo solicitado. image

  3. Por último, se ajustó el package.json según lo planteado en https://github.com/udistrital/planeacion_utilidades_mf/commit/b2cb27067eaf37ed81aa798cbb323ee14f610c15 image

C0C045 commented 5 months ago

PR

seguimiento_mf: https://github.com/udistrital/planeacion_seguimiento_mf/pull/6

milo19980525 commented 5 months ago

@a52290451 y @felimarod por favor revisar el endpoint "/seguimiento/get_seguimiento/{plan_id}/{index}/{trimestre} (planeacion_mid)" con base en el comentario https://github.com/udistrital/planeacion_cliente/issues/1013#issuecomment-2087514762, dado que el mismo endpoint en el api_mid funciona correctamente.

Gracias.

milo19980525 commented 5 months ago

@C0C045 por favor corregir las variables relacionadas con los nuevos micro mids, estás deben pasar por autenticación no por la intranet: https://github.com/udistrital/planeacion_seguimiento_mf/blob/feature/refactor/src/environments/environment.ts.

Por ejemplo: planeacion_seguimiento_mid https://autenticacion.portaloas.udistrital.edu.co/apioas/planeacion_seguimiento_mid/v1 no pruebasapi3....

De igual manera en el archivo de variables de entorno enviroment.ts (y demás archivos) no se encuentra precisamente el api de planeacion_seguimiento_mid, por favor revisar y actualizar.

image

C0C045 commented 5 months ago

PR VARIABLES DE ENTORNO

seguimiento_mf: https://github.com/udistrital/planeacion_seguimiento_mf/pull/7

milo19980525 commented 5 months ago

@C0C045 Perfecto, muy buen trabajo!

a52290451 commented 5 months ago

Se realiza ajuste solicitado en https://github.com/udistrital/planeacion_cliente/issues/1013#issuecomment-2089321101:

udistrital/planeacion_seguimiento_mid/pull/11

udistrital/planeacion_cliente/issues/972

El endpoint referenciado queda funcional.