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

Feat(translation) Adding fr-FR translation. #1196

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Lilly3252
Copy link

No description provided.

@vercel
Copy link

vercel bot commented Oct 13, 2023

@Lilly3252 is attempting to deploy a commit to the discordjs Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Oct 13, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
yuudachi-report ⬜️ Ignored (Inspect) Visit Preview Oct 26, 2023 2:31am

@Lilly3252 Lilly3252 changed the title (Feat.) Adding fr-FR translation. Feat(translation) Adding fr-FR translation. Oct 13, 2023
corrected some translations that i missed
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Shouldn’t it be … sur ce serveur

@Lilly3252
Copy link
Author

Shouldn’t it be … sur ce serveur

Where ?

@ghost
Copy link

ghost commented Oct 26, 2023

Shouldn’t it be … sur ce serveur

Where ?

on line 19. I accidentally clicked the wrong button.

sur ce serveur seems like a more elegant/appropriate way of saying it compared to guilde

"Compte suspect d'arnaque ou de spam",
"Hors sujet en support",
"NSFW",
"Post de merde",
Copy link

Choose a reason for hiding this comment

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

The correct translation for this would be Commentaire de peu de valeur

"show_history": "Afficher l'historique de l'utilisateur {{- user}}"
},
"buttons": {
"goto": "Aller au cas {{- case}}"
Copy link

Choose a reason for hiding this comment

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

Aller à l’infraction {{- case}} would seem better + the e of case is missing

"errors": {
"missing_permissions": "Autorisations manquantes pour kicker {{- user}}"
},
"pending": "Veux-tu vraiment kicker {{- user}}?",
Copy link

Choose a reason for hiding this comment

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

vous should be used instead of tu to match others

},
"cancel": "Softban annulé le {{- user}}",
"success": {
"regular": "softban exécuter avec succès {{- user}}",
Copy link

Choose a reason for hiding this comment

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

not infinitive, should be exécuté

"errors": {
"missing_permissions": "Autorisations manquantes pour verrouiller {{- channel}}",
"already_locked": "{{- channel}} est déjà verrouillé.",
"bot_requires_admin": "Le bot nécessite l'autorisation « ADMINISTRATEUR » pour verrouiller les chaînes, car Discord ne permet pas de définir tous les changements sans cette autorisation, ce qui peut entraîner des problèmes lors du verrouillage ou du déverrouillage."
Copy link

Choose a reason for hiding this comment

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

bot should be changed to robot to match other places

"title": "**Résultats**",
"total": "**Total :** `{{count}}`",
"preliminary": {
"banned_one": "`{{count}}` le membre peut être banni",
Copy link

Choose a reason for hiding this comment

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

no article needed. les should be removed

"total": "**Total :** `{{count}}`",
"preliminary": {
"banned_one": "`{{count}}` le membre peut être banni",
"banned_many": "`{{count}}` les membres peuvent être bannis",
Copy link

Choose a reason for hiding this comment

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

no article needed. les should be removed

"preliminary": {
"banned_one": "`{{count}}` le membre peut être banni",
"banned_many": "`{{count}}` les membres peuvent être bannis",
"banned_other": "`{{count}}` les membres peuvent être bannis",
Copy link

Choose a reason for hiding this comment

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

no article needed. les should be removed

"banned_one": "`{{count}}` le membre peut être banni",
"banned_many": "`{{count}}` les membres peuvent être bannis",
"banned_other": "`{{count}}` les membres peuvent être bannis",
"failed_one": "`{{count}}` Le membre ne peut pas être banni",
Copy link

Choose a reason for hiding this comment

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

no article needed. les should be removed

"failed_many": "`{{count}}` Les membres ne peuvent pas être bannis",
"failed_other": "`{{count}}` Les membres ne peuvent pas être bannis"
},
"blast": {
Copy link

Choose a reason for hiding this comment

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

for all of that: no article needed. les should be removed

@ghost
Copy link

ghost commented Oct 26, 2023

cc @Lilly3252

@Lilly3252
Copy link
Author

Oh , thanks for pointing all those out.
For your question about google translate , the answer is No , i do speak in french fluently as first language.
QC-fr ( my native ) and FR from france is not quite the same , so yea sometimes i did have difficulty to find the good translation cuz some words that were in english that needed to be translated in french ... well ... we use those in english here lol.

  • after sometimes i was quite forgetting the "context" of the sentence if i can say it like that. so yeah there was a moment where i was just translated almost 1:1 and after i was done with the file ( that took like more than a week lol ) i repassed to check if i did mistakes and since theres a LOT of sentences well... i've forgot a lot of it too XD!! haha

I will re-check the whole file. thanks

@ghost
Copy link

ghost commented Oct 26, 2023

I understand your struggle 🤣 My native language is also QC-french. But everything looks good in the file now!

@ghost
Copy link

ghost commented Oct 26, 2023

@iCrawl / @almostSouji Can y’all review this PR?

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.

1 participant