Comentarios

[dani-fmena]

Existen algunos lugares donde se agradecería mucho un comentario para extender y mantener el repo en un futuro. Por ejemplo:

• academic_certificate.ui/src/views/ViewDashboard.vue

  Line 103 in 419e620

  const componentKey = ref(0);

• academic_certificate.ui/src/views/auth/ViewLogin.vue

  Line 75 in 419e620

  const componentKey = ref(0);

  
  Poner la utilidad de la variable, ya que no es obvio uso ni explícito el nombre.

• Comenta los campos que no sean obvios de ICertificatesRow, brevemente la utilidad que tiene, etc. Ej el campo certification, pero es útil comentarlos brevemente todos para no dejar espacio a dudas a la persona que tenga que trabajar con el repo.

• academic_certificate.ui/src/services/helpers/help-forms.ts

  Line 28 in 419e620

  export function transformCertificateResponse (records:any): Array<ICertificatesRow> {

• academic_certificate.ui/src/services/helpers/help-forms.ts

  Line 56 in 419e620

  export function transformCertificateForm (record:ICertificateDto): any {

comenta el objetivo de las funciones, donde se deben usar y pq se tuvo q implementar

• academic_certificate.ui/src/views/certificates/ViewFormCertificates.vue

  Line 444 in 419e620

  const loading = ref(false);

• academic_certificate.ui/src/views/certificates/ViewFormCertificates.vue

  Line 491 in 419e620

  if (cmptdFmode.value === 'edit' as TFormMode || cmptdFmode.value === 'details' as TFormMode) {

  
  comenta esta lógica, brevemente pq hacen falta estas estructuras de control, queda bien en cada rama de cual es el motivo de la misma

• academic_certificate.ui/src/views/certificates/ViewFormCertificates.vue

  Line 543 in 419e620

  if(!['create','edit','details','validate','invalidate'].includes(cmptdFmode.value as TFormMode))

  
  comenta esta lógica, brevemente pq hacen falta estas estructuras de control, queda bien en cada rama de cual es el motivo de la misma

• academic_certificate.ui/src/views/certificates/ViewListCertificates.vue

  Line 120 in 419e620

  if(+searchtype === SearchTypes.ToValidate)

  
  comenta esta lógica, brevemente pq hacen falta estas estructuras de control, queda bien en cada rama de cual es el motivo de la misma