Skip to content

French translation improvements from Crowdin made by:#1660

Merged
magul merged 4 commits intoDjangoGirls:masterfrom
magul:crowdin-translation-fr
May 16, 2020
Merged

French translation improvements from Crowdin made by:#1660
magul merged 4 commits intoDjangoGirls:masterfrom
magul:crowdin-translation-fr

Conversation

@magul
Copy link
Copy Markdown
Member

@magul magul commented May 8, 2020

@magul magul requested a review from a team May 8, 2020 18:52
Comment thread fr/django_forms/README.md Outdated
## Éditer un formulaire

Maintenant, nous savons comment ajouter un nouveau formulaire. Comment faire si nous voulons éditer un formulaire déjà existant ? C'est très similaire à ce que nous venons de faire. Tout d'abord, faisons rapidement quelques choses importantes. (Si vous ne comprenez pas quelque chose, vous devriez demander à votre entraîneur ou regarder les chapitres précédents, car nous avons déjà traité toutes ces étapes.)
Maintenant, nous savons comme ajouter un nouveau post. Comment faire si nous voulons éditer un formulaire déjà existant ? C'est très similaire à ce que nous venons de faire. Tout d'abord, faisons rapidement quelques choses importantes. (Si vous ne comprenez pas quelque chose, vous devriez demander à votre entraîneur ou regarder les chapitres précédents, car nous avons déjà traité toutes ces étapes.)
Copy link
Copy Markdown
Member

@ewjoachim ewjoachim May 8, 2020

Choose a reason for hiding this comment

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

Hm, le changement "comment" > "comme" paraît incorrect, non ?
Quitte à changer "formulaire" pour "post", cela serait plus clair en le changeant également à la phrase suivante.

Suggested change
Maintenant, nous savons comme ajouter un nouveau post. Comment faire si nous voulons éditer un formulaire déjà existant ? C'est très similaire à ce que nous venons de faire. Tout d'abord, faisons rapidement quelques choses importantes. (Si vous ne comprenez pas quelque chose, vous devriez demander à votre entraîneur ou regarder les chapitres précédents, car nous avons déjà traité toutes ces étapes.)
Maintenant, nous savons comment ajouter un nouveau post. Comment faire si nous voulons éditer un post déjà existant ? C'est très similaire à ce que nous venons de faire. Tout d'abord, faisons rapidement quelques choses importantes. (Si vous ne comprenez pas quelque chose, vous devriez demander à votre entraîneur ou regarder les chapitres précédents, car nous avons déjà traité toutes ces étapes.)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Je plaide coupable pour «comme», c’est une erreur de frappe/d’inattention.

Pour le changement de formulaire pour post, je n’ai fait qu’être fidèle à l’original (qui utilise «post» pour cette phrase et «form» pour celles avant et après), je trouve ça plutôt étrange aussi mais je n’ai pas vraiment eu le courage de me replonger dans le contexte du tutoriel pour vérifier.

Cette phrase était marqué comme non-traduite dans CrowdIn donc j’imagine que le texte original utilisait «form» précédemment comme le reste du texte, du coup j’ai fait confiance au changement en anglais/à l’original.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hm, l'original dit actuellement "Now we know how to add a new post. But what if we want to edit an existing one?" du coup, il n'est pas ambigu. Je vote pour une traduction non ambigüe aussi, au choix:

  • Maintenant, nous savons comment ajouter un nouveau post. Comment faire si nous voulons éditer un post déjà existant ?
  • Maintenant, nous savons comment ajouter un nouveau post. Comment faire pour en modifier un qui existe déjà ?

Copy link
Copy Markdown

@ariasuni ariasuni May 9, 2020

Choose a reason for hiding this comment

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

En effet, je n’avais pas vérifier la suite qui était déjà marqué comme traduit. Je pense que la 2e est bien, c’est assez clair sans répéter.

Édit.: je dois faire quelque chose ou tu t’en occupes?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Je peux faire le changement dans crowdin mais je connais pas le process pour faire une PR derrière. En tout cas, je sais pas si faire la modif côté github suffit ou si elle va se faire ecraser par crowdin (quelque chose me dit que c'est la 2e solution).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Capture d’écran 2020-05-09 à 20 57 12

Voila

Copy link
Copy Markdown
Member

@ewjoachim ewjoachim left a comment

Choose a reason for hiding this comment

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

Le reste a l'air 👌
The rest seems 👌

@ewjoachim
Copy link
Copy Markdown
Member

@magul We've updated crowdin to fix a mistake, but not sure how to update this branch. I believe it's in your hands now :)

@magul
Copy link
Copy Markdown
Member Author

magul commented May 11, 2020

@ewjoachim I will update branch now from Crowding and will wait for your approval.

@magul
Copy link
Copy Markdown
Member Author

magul commented May 16, 2020

@ewjoachim @ariasuni Hi there, because of the limitations of our translation workflow we should apply these suggestions to Crowdin project (otherwise every refresh from Crowding will effectively overwrite it here). Can you, please, apply them there and poke me then? I will refresh this pull request with most current state from Crwodin? Or should I apply them there?

Thanks in advance!

@magul
Copy link
Copy Markdown
Member Author

magul commented May 16, 2020

LOL, I lost a track of previous conversation here. Please ignore my previous comment. Can you confirm, that we're ready to merge this translation?

@ewjoachim
Copy link
Copy Markdown
Member

Ah damn, another fix missing. I'll do it right away.

@ewjoachim
Copy link
Copy Markdown
Member

Crowdin updated again. Sorry about that.

@magul
Copy link
Copy Markdown
Member Author

magul commented May 16, 2020

Done @ewjoachim

@magul magul merged commit 94375e5 into DjangoGirls:master May 16, 2020
@magul magul deleted the crowdin-translation-fr branch May 16, 2020 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants