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

Mejora del aviso de "Eliminar relaciones previas" #4612

Closed
contidos opened this issue Feb 16, 2023 · 6 comments
Closed

Mejora del aviso de "Eliminar relaciones previas" #4612

contidos opened this issue Feb 16, 2023 · 6 comments
Milestone

Comments

@contidos
Copy link

En la sección de administración >> Sesiones >> Importar lista de directores RRHH en las sesiones aparace un botón para seleccionar la ubicación del archivo a importar y justo debajo un checkbox que pone "Eliminar relaciones previas".

Podría ser una buena idea agregar un texto más extenso en esa zona que deje más claro para el usuario de a pié qué es exactamente lo que va a hacer ese botón. Yo sugiero lo siguiente:

Eliminar relaciones previas (Se eliminará el usuario RRHH de todas las sesiones que tiene asignadas actualmente)

Y también sería ideal, si fuese posible, que si la persona selecciona el checbox, le salga un mensaje pidiéndole que confirme si está seguro de que quiere eliminar al usuario RRHH de todas las sesiones en las que está inscrito actualmente.

Esta propuesta de mejora viene tras observar el comportamiento de varios usuarios que por error marcan la casilla y con esto quitan al usuario RRHH de las sesiones que ya ha realizado hasta esa fecha,

@ywarnier
Copy link
Member

ywarnier commented Mar 4, 2023

Para la parte "mejorar el aviso" creo que está bien ahora (no use tu texto sino uno un poco más largo).
Para la parte de la validación con un mensaje que le aparece, esto tendrá que esperar otra versión.

@ywarnier
Copy link
Member

ywarnier commented Mar 4, 2023

Capture d’écran du 2023-03-04 01-34-00

Quizás haya una pequeña mejora que hacer en "lo que no existe". Ya me dirás...

@ywarnier ywarnier added this to the 1.11.20 milestone Mar 4, 2023
@contidos
Copy link
Author

contidos commented Mar 6, 2023

Yo lo sigo viendo un poco complejo de entender por el usuario. Esa sección está disponible sólo para cargar de manera masiva el perfil de rrhh, por lo que yo creo que el texto podría hacer referencia a ellos sin problema. Propongo:

"Marcando esta casilla, eliminas todas las asignaciones previas del responsable de recursos humanos en otras sesiones o cursos. Si lo que quieres es conservar las relaciones que ya tiene ese perfil en tu campus, NO marques esta casilla".

¿Cómo lo ves?

@ywarnier
Copy link
Member

ywarnier commented Mar 6, 2023

Está bien

@ywarnier
Copy link
Member

Actualizado en translate. Se añadirá a Chamilo antes de la 1.11.20

@ywarnier
Copy link
Member

ywarnier commented Jun 6, 2023

Fixed and integrated in 1e0231f

@ywarnier ywarnier closed this as completed Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants