Skip to content

fix: Agregar Validacion de permisos en las funcionalidades faltantes#176

Merged
DiegoAlfaro1 merged 2 commits into
MBI-1from
fix/NHF_no_ref_permisos
Jun 7, 2025
Merged

fix: Agregar Validacion de permisos en las funcionalidades faltantes#176
DiegoAlfaro1 merged 2 commits into
MBI-1from
fix/NHF_no_ref_permisos

Conversation

@NicoH00d
Copy link
Copy Markdown
Contributor

@NicoH00d NicoH00d commented Jun 7, 2025

Plantilla PR FrontEnd TEXT & LINES


Descripción

Se agregaron los permisos faltantes a las funcionalidades del sistema

Tipo de cambio

  • Corrección de error (cambio no disruptivo que soluciona un problema)
  • Nueva función (cambio no disruptivo que agrega funcionalidad)
  • Cambio disruptivo (corrección o función que afecta la compatibilidad existente)
  • Este cambio requiere una actualización en la documentación
  • Cambio mínimo (visual o de bajo impacto, sin afectar lógica)

Nueva funcionalidad (si aplica)

  • Repositorio
  • Modelo
  • Hook
  • Rutas y Rutas Individuales
  • Pagina

Si creaste algo más, ponlo aquí:

  • Ejemplo.jsx

¿Cómo se ha probado?

  • Se utilizó lint para errores
  • Pasaron las pruebas unitarias (si aplica)
  • Se probó la funcionalidad y no mostró errores
  • Visualmente cumple con estándares definidos en el Figma

Cambios menores

  • Este PR realiza un cambio mínimo que no afecta la lógica del sistema
  • Se validó visualmente el componente afectado
  • No se realizaron pruebas unitarias porque no aplica

Checklist del evaluador:

  • He determinado que el autor cumplió con todos los puntos mencionados
  • He determinado que los cambios no tienen un impacto negativo en la aplicación
  • He notificado al autor del PR en caso de dudas o ajustes necesarios
  • Una vez que acepte el PR, eliminaré la rama

@DiegoAlfaro1 DiegoAlfaro1 merged commit c8c261d into MBI-1 Jun 7, 2025
2 checks passed
@DiegoAlfaro1 DiegoAlfaro1 deleted the fix/NHF_no_ref_permisos branch June 7, 2025 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants