Translation "email_intro" is not working in email about decidim HOT 5 CLOSED

@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 😄

from decidim.

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 !

from decidim.

@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

from decidim.

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

from decidim.

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

from decidim.