Fix "people" label in contextual room menu

[odelcroi]

In room context menu, label was "Direct Messages" where it should be "People"

| Personnes 2 |

[NicolasBuquet]

En lisant ton descriptif, il me semble que tu veux remplacer "Direct Messages" par "People".

En lisant le patch j'ai l'impression que c'est l'inverse qui est fait : remplacer "People" par "Direct Messages" ?

[odelcroi]

Non, ce qui est fait est d'enlever le remplacement de "Personne" par "Direct Messages". 💇

[NicolasBuquet]

Avec cette modification :

-                label={_t("People")}
+                // label={TCHAP: change label _t("People")}
+                label={_t("Direct Messages") /* end TCHAP */}

pour moi, à la fin on reste avec label={_t("Direct Messages") /* end TCHAP */} et donc le label basé sur Direct Messages et non People.

[mcalinghee]

In room context menu, label was "Direct Messages" where it should be "People"

je n'ai pas bien compris :

• dans la description : on veut remplacer la cle "Direct Messages" par "People"
• dans la PR/code : on veut remplacer la cle "People" par "Direct Messages"

[odelcroi]

Oui, c'est tricky. Il faut lire :

- -                label={_t("People")}
- +                // label={TCHAP: change label _t("People")}
- +                label={_t("Direct Messages") /* end TCHAP */}

C'est à dire que dans la PR, les lignes de patch sont enlevées, donc on retourne à l'état inital

label={_t("People")}

[mcalinghee]

en fait on a les "-" partout et du coup on enleve tout 😅

[NicolasBuquet]

Ah ok, c'est un diff de diff.

J'avais raté les '-' partout !

Ok !