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

Feat-Modif. Front formacionesPosgrado #206

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Feat-Modif. Front formacionesPosgrado #206

wants to merge 2 commits into from

Conversation

JuanIRamirez
Copy link
Contributor

@JuanIRamirez JuanIRamirez commented Apr 24, 2023

Requerimiento

https://proyectos.andes.gob.ar/browse/MAT-137

Funcionalidad desarrollada

  1. Se modificaron los componentes de Detalle profesional, Alta , Editar y Detalle Especialidad de formacionesPosgrado.
  2. Se modificó interface IProfesional.
  3. Antes de mergear correr el script MAT-134 y MAT-119 para corregir la colección profesional.

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

  • Si
  • No

@JuanIRamirez JuanIRamirez requested review from a team as code owners April 24, 2023 15:52
@JuanIRamirez JuanIRamirez changed the title Feat( Modif. Front formacionesPosgrado) Feat-Modif. Front formacionesPosgrado Apr 25, 2023
@silviroa
Copy link
Contributor

USUARIO: jramirez
BUILD NUMBER: 8757
CYPRESS RUN: 7930
TEST START: 2023-04-25T15:15:21.960Z
TOTAL: 386
SUCCESS: 378
FAIL: 6
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: jramirez
BUILD NUMBER: 8758
CYPRESS RUN: 7931
TEST START: 2023-04-25T17:40:38.483Z
TOTAL: 406
SUCCESS: 402
FAIL: 2
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: jramirez
BUILD NUMBER: 8759
CYPRESS RUN: 7932
TEST START: 2023-04-25T18:44:10.032Z
TOTAL: 406
SUCCESS: 404
FAIL: 0
SKIPPED: 2

@JuanIRamirez JuanIRamirez force-pushed the MAT-137 branch 3 times, most recently from cf4de4a to 6070343 Compare May 10, 2023 17:41
@MarianoCampetella
Copy link
Contributor

image

Una ves ejecutado los scripts necesarios para poder comenzar a probar el pr se encontraron los siguientes problemas:

  • Las matriculas de posGrados pasan a quedar suspendidas cuando se encontraban en otro estado (vigente, vencida, ...)
  • En el dropdown de los tres puntitos que esta al lado del bagde del estado me sale solamente la opción de "editar" cuando deberia salir "desactivar vencimiento" y "dar de baja". Ademas el ítem de editar en tareas previas se había quitado.
  • Cuando seleccionamos el botón de "ver profesional" el cual nos dirige a una nueva vista, automáticamente nos arroja un modal con un error seguido de varios problemas que figuran en la consola.
    image

@@ -16,15 +16,23 @@ export class FormacionPosgradoAgregarComponent implements OnInit {

public cancel = false;
profesiones: any[] = [];
vencimientoAnio = (new Date()).getUTCFullYear() + 5;
profesionalP: any = {
fechaAlta = new Date();
Copy link
Contributor

Choose a reason for hiding this comment

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

Ya que estamos trabajando con la libreria moment() podriamos cambiar esa asignacion por fechaAlta = moment()
En caso de que sea necesario trabajarla en formato Date se puede asignar como fechaAlta = moment().toDate()

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Reemplacé 'new Date()' p/ 'moment().toDate()'

</td>
</tr>
</plex-table>
<div *ngIf="!editarObtencion && !editarRevalida && !altaRevalida && !altaObtencion">
Copy link
Contributor

Choose a reason for hiding this comment

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

Al igual que en todos los *ngIf="..." cuando son bastantes extensos ya que se realizan multiples verificaciones, por buenas practicas, lo ideal seria armar una función en el .ts, llamarlo desde el html y que devuelva true o false si cumple con todas esas condiciones.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cambié '*ngIf="!editarObtencion && !editarRevalida && !altaRevalida && !altaObtencion"' por funcion 'obtenerMatricula()'

* feat(MOBILE): configuracion de parametros con qr desde mobile

* fix(MOBILE):Responsive listado guia profesional

* ref(MOBILE): oculta filtros cuando viene la busqueda por parametro

---------

Co-authored-by: aldoM <[email protected]>
Co-authored-by: Lautaro Molina <[email protected]>
@JuanIRamirez
Copy link
Contributor Author

Cuando arroja el modal con un error de conexiòn es por la VPN. no conectada.
Errores de consola corregidos.

@MarianoCampetella
Copy link
Contributor

Juan cuando corro el script sobre mi base local, con varios profesionales sacados de demo que están cargados desde el año 2020 hasta la actualidad me ocurre lo siguiente de la imagen. Mas allá que con los profesionales nuevos o los que fueron creados en este año no ocurra, hay que contemplar múltiples casos que pueden llegar a darse. En todo caso traete este profesional 35493155 de la base de demo y realiza múltiples pruebas para evitar que cuando se corra en producción existan estos problemas. Cualquier cosa lo vamos trabajando juntos para que ya quede listo!
Saludos!
image

@MarianoCampetella MarianoCampetella added changes requested and removed changes done Cambios realizados labels Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants