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

Translation "email_intro" is not working in email #50

Closed
juliesimon opened this issue Mar 12, 2018 · 5 comments
Closed

Translation "email_intro" is not working in email #50

juliesimon opened this issue Mar 12, 2018 · 5 comments
Assignees
Labels
translation translation error/enhancement

Comments

@juliesimon
Copy link

On pompano.osp.cat, when there is a new comment, the mail that is sent to the administrator is not using the french translation for the email_intro variable :
capture d ecran 2018-03-12 a 14 02 25

But we can see in the locale file decidim/decidim_comments/config/locales/fr.yml that email_intro is translated in french

    events:
      comments:
        comment_created:
          comment:
            email_intro: '"%{resource_title}" a été commenté. Vous pouvez lire le commentaire sur cette page:'
            email_outro: Vous avez reçu cette notification parce que vous suivez "%{resource_title}" ou son auteur. Vous pouvez annuler ces notifications à partir du lien précédent.
            email_subject: Un nouveau commentaire a été publié par %{author_name} dans %{resource_title}
            notification_title: Un nouveau commentaire a été publié par <a href="%{author_path}">%{author_name} %{author_nickname}</a> dans <a href="%{resource_path}">%{resource_title}</a>
@juliesimon juliesimon added translation translation error/enhancement 0.9-stable labels Mar 12, 2018
@mrcasals
Copy link

@juliesimon for some reason, the code is looking for decidim.comments.events.comment_created, while the YAML file has decidim.events.comments.comment_created (scopes are changed).

You'll need to fix the keys order so that is works 😄

@juliesimon
Copy link
Author

Hi @ludivinecp @moustachu

Suite à cette réponse, j'ai fouillé un peu plus en longueur le dossier decidim-comments de notre branche 0.9-stable, en le comparant avec celui des barcelonais :
Dans le fichier de locales d'osp , osp/decidim/decidim_comments/config/locales/en.yml - lignes 4 à 26 - il y a tout un "jeu" de locales en plus par rapport à decidim/decidim/decidim_comments/config/locales/en.yml de decidim/decidim#0.9-stable.
D'après ce que j'ai compris, c'est que ces locales ont été ajoutées par OSP pour être utilisées dans le fichier comment_authorized_event.rb (fichier qui n'existe pas chez les barcelonais), en plus des locales utilisées également dans comment_created_event_extand.rb
Mais visiblement à un moment donné il y a un cafouillage entre les appels de ces locales, car les traductions ne passent plus.
J'ai essayé de trouver d'où vient le problème, il faut probablement modifier l'ordre des clefs dans un des appels de locale mais je ne sais pas de quel fichier vient la méthode appelée, si c'est celle qui est dans comment_created_event_extand.rb ou dans comment_authorized_event.rb.
Et surtout, je ne comprends pas pourquoi on a les même traductions définies deux fois dans le même fichier.

J'espère que c'est un peu clair quand même.
J'attends votre avis la dessus !

@ludivinecp
Copy link

@juliesimon c'est provoqué en effet par la modération.
La méthode se trouve dans comment_created_event_extend.rb -> ligne 5
Et les trads sont dans le fichier fr.yml ligne 90. Je pense que ce sont celles ci les trads que tu penses en double. Elles sont en fait spécifiques à la modération. Je les aient mises en bas afin d'éviter les conflits lors des MAJ

@juliesimon
Copy link
Author

@ludivinecp ok ! Et donc tu as résolu l'issue ? Il faut que je teste ?

@ludivinecp
Copy link

@juliesimon non je te donnais les clés pour que tu puisses la résoudre sans trop de mal ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation translation error/enhancement
Projects
None yet
Development

No branches or pull requests

3 participants