Skip to content

Update fr_FR_glossary.json - #36

Closed
Flobul wants to merge 2 commits into
Mips2648:mainfrom
Flobul:patch-3
Closed

Flobul wants to merge 2 commits into
Mips2648:mainfrom
Flobul:patch-3

Conversation

@Flobul

@Flobul Flobul commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Add "parti" as "left" instead of "party"

@Mips2648

Mips2648 commented Aug 27, 2026 •

Copy link
Copy Markdown
Owner

I don't think this request is a good idea:, it is a glossary, not a dictionary so:

  • We should not add conjugated forms
  • we should not translate in all languages if there is no need

on top, I don't see in which context the word "parti" (alone) should not be translated as "party" but "left", so why adding a glossary item for that?
it is even more weird that if it should be translated as "left", it means the word is included in a sentence and so deepl would never suggest to use "party" examples:

  • "il est parti" will be translated to "he left" by deepl, so it is already correct
  • but "un parti politique" will be (and must be) translated to "a political party" but with your proposal, the final translation would be "a political left" which is completely wrong!

so please provide use case to reproduce first

@Flobul

Flobul commented Aug 27, 2026 •

Copy link
Copy Markdown
Contributor Author

I get it now.

But in my case, "Parti" is alone :

                                $enumValues = array(
                                    array('value'=>'Default','text'=>__('Défaut',__FILE__)),
                                    array('value'=>'Normal','text'=>__('Normal',__FILE__)),
                                    array('value'=>'Away','text'=>__('Parti',__FILE__)),
                                    array('value'=>'Vacation','text'=>__('Vacances',__FILE__)),
                                    array('value'=>'Home','text'=>__('Maison',__FILE__))
                                );

I think the glossary is not helpful in that case.

Eventually, I should force the right translation manually in any language so it won't be overwritten.

In conclusion, isolated ambiguous strings must be manually pre-translated in the i18n files, not add in glossary.

@Mips2648

Copy link
Copy Markdown
Owner

In my opinion "Parti" is incorrect here, as it does not mean ‘Away’ in French. It sounds a bit strange to me because it is not a noun but a (conjugated) verb, unlike the others, so doesn't seems right to use it here.
I would have chosen: "Absent" (because it is short) or "A l'extérieur" ou "Pas à la maison";
"Absent" is also commonly used in such "security app".

And with such word you wouldn't have the translation issue ;-) ... but maybe another so anyway, forcing a translation in the i18n file is indeed to best in such use case.

@Flobul Flobul closed this Sep 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants