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

Hotfix - Personas y Organizaciones - Condición para aviso cuando email masivo #504

Merged
merged 2 commits into from
Dec 19, 2024

Conversation

enricsinergia
Copy link
Collaborator

Descripción

En #418 se introdujo un mensaje de aviso para que el usuario sea consciente que si intenta enviar un mensaje de correo después de utilizar la opción "seleccionar todo", en realidad sólo se envía el mensaje a las personas / Organizaciones de la página actual.
Sin embargo la condición utilizada para detectar que se ha escogido la opción "todos los registros" ha dado algunos problemas. Se cambia esta condición después de revisar cómo setea ese flag la función check_entire_list

Pruebas

En las vistas de Personas / Organizaciones / Interesados:

  • Sin marcar ningún filtro ni paginar, escoger la opción Todos los registros en selección y dentro de acciones masivas la opción correo electrónico. Comprobar que aparece el mensaje de advertencia
  • Repetir la prueba tras usar algún filtro y paginar el resultado
  • Probar a chequear la opción, deschequearla, chequear otra, paginar, volver, .... y comprobar que el mensaje aparece cuando debe y no aparece cuando no debe

@enricsinergia enricsinergia marked this pull request as ready for review December 4, 2024 11:49
Copy link

github-actions bot commented Dec 4, 2024

Actions executed at: 2024-12-19 08:19:04.

Copy link
Collaborator

@jordiSTIC jordiSTIC left a comment

Choose a reason for hiding this comment

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

(A)Probado.

Como observación: el mensaje aparece siempre que se envie correo seleccionando "Todos los registros", aunque el número de registros totales sea menor que el de la página actual. Por ejemplo: aplicando un filtro que obtenga un único registro

Copy link
Collaborator

@ManuSinergiaCRM ManuSinergiaCRM left a comment

Choose a reason for hiding this comment

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

(A)probado

@AlbertoSTIC AlbertoSTIC merged commit 0464c4a into develop Dec 19, 2024
1 check passed
@AlbertoSTIC AlbertoSTIC deleted the bug/mailWarningPages branch December 19, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incidencia - Personas y Organizaciones - Mensaje de sólo se enviará a la página actual no siempre aparece
4 participants