Skip to content

Fix: Correcciones Frontend#206

Merged
carlex74 merged 5 commits into
developfrom
fix/auth-practices-and-interface
Jan 11, 2026
Merged

Fix: Correcciones Frontend#206
carlex74 merged 5 commits into
developfrom
fix/auth-practices-and-interface

Conversation

@LucaTvl
Copy link
Copy Markdown
Collaborator

@LucaTvl LucaTvl commented Jan 5, 2026

Issue Relacionada

Closes #205

Resumen

Este PR incluye mejoras importantes en la arquitectura del proyecto con 5 commits enfocados en refactorización de componentes, estandarización del modelo Professor, resolución de problemas de seguridad en headers HTTP, y actualización de dependencias vulnerables. Se mejora la reutilización de código y se fortalece la seguridad de la API.

Cambios Técnicos Implementados

  • Seguridad - API Client: Se eliminó la duplicación del header Authorization en el interceptor de solicitudes para evitar conflictos con la configuración de credenciales.

  • Refactorización de Componentes: Se extrajo el componente PageLoader a un archivo independiente, mejorando la modularidad y permitiendo su reutilización en múltiples partes de la aplicación.

  • Refactorización del Modelo Professor: Se reemplazaron funciones auxiliares del profesor con métodos e propiedades en la clase Professor, centralizando la lógica de negocio y mejorando la mantenibilidad. Se agregaron propiedades calculadas como fullName, profilePicture, email y initials.

  • Limpieza de Modelos: Se removieron propiedades obsoletas de JoinInstitutionModal y Professor model, eliminando campos no utilizados que generaban deuda técnica.

Guía para Pruebas y Revisión

  • Clonar y instalar dependencias:
pnpm install
  • Validar cambios en API Client:

  • Iniciar el servidor:

pnpm run dev
  • Verificar que las solicitudes autenticadas incluyan solo un header Authorization
  • Usar DevTools (Network tab) para confirmar que no hay headers duplicados

Probar componente PageLoader:

  • Navegar a páginas que usen PageLoader
  • Verificar que el estado de carga se muestre correctamente
  • Confirmar que no hay errores de importación

Probar modelo Professor:

  • Acceder a secciones donde se use el objeto [Professor]
  • Verificar que los getters fullName, profilePicture, email y initials. retornan valores correctos

@LucaTvl LucaTvl requested review from NiconiKimg and carlex74 January 5, 2026 17:34
@LucaTvl LucaTvl added bugfix Fix existing bugs, errors, or unexpected behavior in the project refactor Improve the code structure, readability, or performance without changing its external behavior labels Jan 5, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
front-end-dsw Ready Ready Preview, Comment Jan 5, 2026 5:34pm

@LucaTvl LucaTvl changed the title Fix: Auth practices and interface Fix: Correcciones Frontend Jan 5, 2026
@carlex74 carlex74 merged commit ad9c80a into develop Jan 11, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fix existing bugs, errors, or unexpected behavior in the project refactor Improve the code structure, readability, or performance without changing its external behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issues Frontend

3 participants