Issues Frontend

[lucasjaviluna]

• Por que usan 2 mecanismos diferentes para setear el header de Authorization?
  

  Front-End-DSW/src/api/apiClient.ts

  Line 85 in 5aa54be

  apiClient.defaults.headers.common['Authorization'] = `Bearer ${newToken}`;

  
  

  Front-End-DSW/src/contexts/AuthProvider.tsx

  Line 79 in 5aa54be

  apiClient.defaults.headers.common['Authorization'] = `Bearer ${token}`;

  
  Veo que están usando el interceptor de Axios, con eso debería ser suficiente para chequear el token y agregar el header dinámicamente en cada request, sin acudir a los header default que son globales y estáticos.

• Siempre es buena práctica tener un archivo solo con la definicion de un componente

  Front-End-DSW/src/router/AppRouter.tsx

  Line 9 in 5aa54be

  const PageLoader = () => (

• Tal vez hubiera sido mejor usar una class en lugar de una interface para Proffesor. De esa manera todas las funciones de Proffesor que tienen como utils, podrían ser métodos de la clase Proffesor directamente (como getProfessorName, getProfessorProfilePicture, etc)
  

  Front-End-DSW/src/lib/professor.ts

  Line 1 in 5aa54be

  import type { Professor } from '../types/entities';