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

RF22 Eliminar Colaboradores #67

Merged
merged 11 commits into from
Apr 29, 2024
Merged

RF22 Eliminar Colaboradores #67

merged 11 commits into from
Apr 29, 2024

Conversation

OlimpiaGarcia
Copy link
Contributor

@OlimpiaGarcia OlimpiaGarcia commented Apr 26, 2024

RF22 Eliminar Colaboradores

[Feature/Task ID]: RF22 Eliminar Colaboradores

Descripción

Descripción clara y concisa de los cambios realizados. Incluye el contexto relevante, como historias de usuario o criterios de aceptación.

Cambios realizados

  • Modal de eliminar empleado se abre y cierra. Sin recargar toda la pagina
  • Se elimina correctmente el empleado

Story Points

Estima la complejidad de la tarea en puntos de historia.

Criterios de aceptación

  • El sistema permite al administrador buscar a empleados por nombre para seleccionarlos para la baja.
  • Al seleccionar un empleado para la baja, el sistema muestra una pantalla de confirmación antes de proceder.
  • Una vez confirmada la baja por el administrador, el sistema elimina al empleado de la plataforma.
  • El sistema proporciona una confirmación visual en la plataforma de que el empleado ha sido dado de baja correctamente.

Dependencias

depende de consultar empleados, ya esta mergeado

Definición de Done

  • El código ha sido revisado por al menos un miembro del equipo.
  • Todas las pruebas pasan localmente.
  • El código sigue los estándares de codificación y las mejores prácticas del proyecto.
  • La documentación ha sido actualizada (si corresponde).
  • Cualquier nueva dependencia está documentada y justificada.
  • Los cambios han sido probados en un entorno de preparación (si corresponde).

ScreenShot de la pagina

imagen
imagen

Copy link

vercel bot commented Apr 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
zeitgeist-front-end ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 29, 2024 6:32pm
zeitgeist-front-end-ru7b ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 29, 2024 6:32pm

src/components/common/DeleteModal.tsx Outdated Show resolved Hide resolved
src/components/modules/Employees/EmployeeeTable.tsx Outdated Show resolved Hide resolved
src/hooks/useDeleteEmployee.ts Outdated Show resolved Hide resolved
Copy link
Member

@salgue441 salgue441 left a comment

Choose a reason for hiding this comment

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

As mentioned, the same files are modified on RF#74 and this PR.

src/components/common/DeleteModal.tsx Show resolved Hide resolved
@DHurtado714-itesm DHurtado714-itesm changed the title Yuna/rf22 RF22 Eliminar Colaboradores Apr 29, 2024
src/utils/constants.ts Show resolved Hide resolved
@DHurtado714-itesm DHurtado714-itesm merged commit 634ef98 into develop Apr 29, 2024
4 checks passed
@dembA7 dembA7 deleted the Yuna/RF22 branch May 9, 2024 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Cerrado
Development

Successfully merging this pull request may close these issues.

5 participants