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

Translate "React Components, Elements, and Instances" page #114

Merged
merged 6 commits into from
Oct 16, 2019

Conversation

forresst
Copy link
Contributor

@forresst forresst commented Mar 19, 2019

100% translated

@netlify
Copy link

netlify bot commented Mar 19, 2019

Deploy preview for fr-reactjs ready!

Built with commit a052e43

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

@tdd
Copy link
Collaborator

tdd commented Apr 26, 2019

@forresst si j'ai bien compris ce truc est prêt depuis le 22/3 mais t'as jamais enlevé le statut brouillon sur la PR, c'est ça ?

@forresst
Copy link
Contributor Author

@tdd Désolé je suis à la bourre, j'ai bientôt terminé la traduction et je dois refaire une repasse, après je changerai le statut de la PR

@forresst forresst marked this pull request as ready for review April 30, 2019 09:26
@forresst
Copy link
Contributor Author

Désolé pour le retard, merci pour vos relectures

@forresst forresst changed the title (WIP) Translate "React Components, Elements, and Instances" page Translate "React Components, Elements, and Instances" page May 3, 2019
Copy link

@Elanis Elanis left a comment

Choose a reason for hiding this comment

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

Bonjour !

J'ai écris quelques petites remarques sur ta traduction.

J'espère que ça aidera :)

@forresst
Copy link
Contributor Author

@Elanis merci pour tes remarques, j'ai fait les corrections

@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Sep 12, 2019
@tdd tdd self-assigned this Sep 12, 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 @forresst !

Je suis sûr que tu n'y comptais plus, mais me revoilà ! 🎉

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)

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

Ce sont surtout des micro-détails, plus deux-trois contre-sens légers.

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 !

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 @forresst !

Je suis sûr que tu n'y comptais plus, mais me revoilà ! 🎉

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)

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

Ce sont surtout des micro-détails, plus deux-trois contre-sens légers.

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 !

@tdd tdd added Reviewed and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Oct 16, 2019
forresst and others added 2 commits October 16, 2019 15:19
@forresst
Copy link
Contributor Author

@tdd Encore une fois merci pour ta relecture
J'ai corrigé et/ou approuvé tes demandes, je te laisse faire la "repasse visuelle"

@tdd tdd merged commit 03e1b24 into reactjs:master Oct 16, 2019
@tdd tdd deleted the react-components-elements-and-instances branch October 16, 2019 13:34
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.

3 participants