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

Enhancement - Encuestas - Mensaje agradecimiento en el idioma seleccionado #339

Merged
merged 5 commits into from
Sep 23, 2024

Conversation

ainaraRT
Copy link
Collaborator

Desarrollo del issue
Al completar una encuesta se redirige a una página dónde el mensaje está únicamente en inglés "Thanks for completing this survey". El mensaje se debería mostrar en el idioma por defecto de la instancia.

Solución implementada:
Se ha implementado las diferentes cadenas con el mensaje que aparecía al principio en inglés, por ello se ha creado una variable con la función de translate() para que recoja dichas cadenas según el idioma seleccionado en el login.

Pruebas

  1. Acceder con cualquier idioma al CRM.
  2. Ir al módulo de encuestas y crear una con alguna pregunta.
  3. Enviar la encuesta y comprobar que aparece el mensaje en el idioma seleccionado en el login o el que tiene por defecto la instancia.

@ainaraRT ainaraRT added enhancement New feature or request Encuestas labels Aug 14, 2024
@ainaraRT ainaraRT self-assigned this Aug 14, 2024
Copy link

github-actions bot commented Aug 14, 2024

Actions executed at: 2024-09-23 14:38:08.

@ainaraRT ainaraRT requested review from ManuSinergiaCRM and jordiSTIC and removed request for enricsinergia and ManuSinergiaCRM August 14, 2024 14:22
@ainaraRT ainaraRT changed the title Enhancement - Encuestas - Mensaje agradecimiento fijado en inglés Enhancement - Encuestas - Mensaje agradecimiento en el idioma seleccionado Aug 14, 2024
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

jordiSTIC
jordiSTIC previously approved these changes Aug 21, 2024
juanSTIC
juanSTIC previously approved these changes Aug 23, 2024
Copy link
Collaborator

@juanSTIC juanSTIC left a comment

Choose a reason for hiding this comment

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

Visto y 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.

Si se modifica el código del core (que es correcto), las cadenas también tienen que ir a los ficheros del core.

@ainaraRT ainaraRT dismissed stale reviews from juanSTIC and jordiSTIC via a6c10a6 September 3, 2024 08:24
@ainaraRT
Copy link
Collaborator Author

ainaraRT commented Sep 3, 2024

Si se modifica el código del core (que es correcto), las cadenas también tienen que ir a los ficheros del core.

Hecho!!

jalbaiges
jalbaiges previously approved these changes Sep 4, 2024
juanSTIC
juanSTIC previously approved these changes Sep 20, 2024
Copy link
Collaborator

@juanSTIC juanSTIC left a comment

Choose a reason for hiding this comment

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

Aprobado

Copy link
Collaborator

@AlbertoSTIC AlbertoSTIC left a comment

Choose a reason for hiding this comment

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

Aprobado

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

@AlbertoSTIC AlbertoSTIC merged commit 79331e0 into develop Sep 23, 2024
1 check passed
@AlbertoSTIC AlbertoSTIC deleted the enhancement/thanksMessageInSurveys branch September 23, 2024 14:37
@SinergiaCRM
Copy link
Collaborator

PR en SA: salesagility#10565

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Encuestas enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mejora - Encuestas - Mensaje agradecimiento fijado en inglés
6 participants