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 - Aviso en acción masiva de envío de correo #418

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

enricsinergia
Copy link
Collaborator

Descripción

En SuiteCRM, cuando en la selección de la vista de lista de Personas u Organizaciones se seleccionan TODOS los registros, en realidad el envío de e-mail sólo se aplica a los registros de la página que se está visualizando.
En lugar de cambiar este comportamiento, que permitiría entonces enviar a gran volumen de Personas sin usar Campañas, se decide poner un alert para avisar de esta circunstancia a quién lo esté intentando hacer y conminandolo a que use el módulo de Campañas.

Pruebas

1.- En la vista de lista de Personas u Organizaciones, seleccionar algunos registros o todos los de la página y observar que al enviar Email (acciones masivas) no aparece aviso alguno
2.- En la misma lista, selccionar TODOS los registros, y comrpobar que al enviar email aparece el aviso

Copy link

github-actions bot commented Oct 3, 2024

Actions executed at: 2024-10-08 09:02:48.

ManuSinergiaCRM
ManuSinergiaCRM previously approved these changes Oct 3, 2024
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

Aprovecho para hacer una propuesta para añadir al principio de la cadena de texto: "Has seleccionado la opción de Seleccionar Todo pero"

A la par, como se tocan cadenas, habría que añadir a Jaume a la revisión

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

Copy link

@jalbaiges jalbaiges left a comment

Choose a reason for hiding this comment

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

Cadenas revisadas. Dado que se trata de una cadena incluida en el fichero de idioma de aplicación y no a nivel de módulo, creo que el nombre interno (LBL_ONLY_CURRENT_PAGE) debería hacer alguna referencia al concepto email. Por ejemplo: LBL_EMAIL_SENDING_ALERT_CURRENT_PAGE. O algo por el estilo.

@enricsinergia
Copy link
Collaborator Author

Cadenas revisadas. Dado que se trata de una cadena incluida en el fichero de idioma de aplicación y no a nivel de módulo, creo que el nombre interno (LBL_ONLY_CURRENT_PAGE) debería hacer alguna referencia al concepto email. Por ejemplo: LBL_EMAIL_SENDING_ALERT_CURRENT_PAGE. O algo por el estilo.

Cambiado

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

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 - General - Acción masiva Correo no incluye todos los destinatarios al "Seleccionar Todo"
4 participants