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

This Is Not the One With Hooks (Traduction complete) #138

Merged
merged 3 commits into from Sep 16, 2019
Merged

This Is Not the One With Hooks (Traduction complete) #138

merged 3 commits into from Sep 16, 2019

Conversation

emmadal
Copy link
Contributor

@emmadal emmadal commented Mar 30, 2019

salut @tdd je viens de pull request la nouvelle page que vous m'avez assigne.

@netlify
Copy link

netlify bot commented Mar 30, 2019

Deploy preview for fr-reactjs ready!

Built with commit db0c04b

https://deploy-preview-138--fr-reactjs.netlify.com

@tdd tdd self-requested a review March 30, 2019 19:22
@tdd tdd self-assigned this Mar 30, 2019
@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Mar 30, 2019
Copy link
Collaborator

@tdd tdd left a comment

Choose a reason for hiding this comment

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

Salut @emmadal !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.

Avant de traiter les retours, lis bien TOUT ce message, notamment les consignes d'intégration optimale des retours par lot(s)

Tout est fait sous forme de Suggestions GitHub, il te suffit à chaque suggestion que tu approuves de
cliquer le bouton d'ajout au lot de modifs à committer (pas de commit direct !), et sur la fin de créer le commit groupé. (ce mode de regroupement en un seul commit est à favoriser absolument, mais tu devras aller dans l'onglet Files changed pour y accéder, il n'est pas utilisable dans l’onglet Conversation)

En gros l'essentiel des soucis entre dans deux catégories :

  • Traductions mot-à-mot, sans prise en compte du contexte tech/docs/React
  • Erreurs d'orthographe et de syntaxe (moins souvent de conjugaison ou de grammaire)

Si certaines appellent discussion, n'hésite pas, tu réponds, on en parle, on avance ensemble 🤝

Quand tout sera bouclé côté code, je ferai une repasse visuelle finale en local avant de fusionner.

Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans master !

content/blog/2018-12-19-react-v-16-7.md Outdated Show resolved Hide resolved
content/blog/2018-12-19-react-v-16-7.md Outdated Show resolved Hide resolved
content/blog/2018-12-19-react-v-16-7.md Outdated Show resolved Hide resolved
content/blog/2018-12-19-react-v-16-7.md Outdated Show resolved Hide resolved
content/blog/2018-12-19-react-v-16-7.md Outdated Show resolved Hide resolved
content/blog/2018-12-19-react-v-16-7.md Outdated Show resolved Hide resolved
content/blog/2018-12-19-react-v-16-7.md Outdated Show resolved Hide resolved
content/blog/2018-12-19-react-v-16-7.md Outdated Show resolved Hide resolved
content/blog/2018-12-19-react-v-16-7.md Outdated Show resolved Hide resolved
content/blog/2018-12-19-react-v-16-7.md Outdated Show resolved Hide resolved
@tdd tdd added Reviewed and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Sep 12, 2019
This Is Not the One With Hooks (Traduction complete) #138
Hello @tdd, jai applique ce que vous m'aviez demande. Desole pour les fautes d'orthographe le soucis est que mon clavier est un QWERTY, raison pour laquelle je fais assez de faute syntaxique et dans l'avenir j'essayerais de ne plus traduitr mot a mot mais plutot dans son contexte

Co-Authored-By: Christophe Porteneuve <tdd@tddsworld.com>
Copy link
Contributor Author

@emmadal emmadal left a comment

Choose a reason for hiding this comment

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

This Is Not the One With Hooks (Traduction complete) #138
Hello @tdd, merci pour vos critiques et corrections je suis desole pour les fautes de syntaxe car mon clavier est un QWERTY mais je vais remedier a cela et aussi dans l'avenir je vais traduire les phrases dans son contexte. Merci car vous me permettez d'avancer grace a vos critiques.
👍

@tdd tdd merged commit 9cce231 into reactjs:master Sep 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants