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

Traduction de la page "Uncontrolled Components" #49

Merged
merged 6 commits into from
Mar 14, 2019

Conversation

LucasPerso
Copy link
Contributor

Bonjour à tous 👋
Voici ma traduction de la page Uncontrolled Components

J'ai également traduit l'exemple input-type-file.js qui est incrusté dans cette page.
Je me suis beaucoup inspiré des traductions de @tlenclos sur la page des Formulaires. Merci à lui !

@LucasPerso LucasPerso force-pushed the uncontrolled-components branch from d44ce1f to b6cbd58 Compare February 17, 2019 17:10
@JeremiePat JeremiePat added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 18, 2019
Copy link
Collaborator

@sheplu sheplu left a comment

Choose a reason for hiding this comment

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

deux ou trois retours / suggestions sur des mots ou des tournures un peu bizarres

content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
@netlify
Copy link

netlify bot commented Feb 24, 2019

Deploy preview for fr-reactjs ready!

Built with commit 6145ea4

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

@LucasPerso
Copy link
Contributor Author

Que puis-je faire ou que manque t-il pour aider à merger cette PR ?

Copy link
Collaborator

@JeremiePat JeremiePat left a comment

Choose a reason for hiding this comment

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

Salut :)

Nickel cette traduction. Je te suggère juste un changement mineur. Il ne reste plus qu'a avoir l'approbation de @tdd pour le merge ;)

content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
@tdd tdd self-assigned this Mar 8, 2019
@tdd tdd added Reviewing Un mainteneur qualifié est en train de composer la revue de code and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Mar 13, 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 @LucasPerso !

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

C'est franchement bien pour une première trad ! 👏 Beaucoup de bonnes formules, de bons choix de phrasé, peu de fautes ortho/syntaxe, etc. On sent que tu as pris le temps de lire les MP et le Glossaire, ça fait plaisir 😁

Naturellement j'ai quelques petites retouches, mais rien de méchant.

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 🤝

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/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
content/docs/uncontrolled-components.md Show resolved Hide resolved
content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
content/docs/uncontrolled-components.md Outdated Show resolved Hide resolved
examples/uncontrolled-components/input-type-file.js Outdated Show resolved Hide resolved
@tdd tdd removed the Reviewing Un mainteneur qualifié est en train de composer la revue de code label Mar 14, 2019
Co-Authored-By: LucasPerso <lucas.personnaz@gmail.com>
@tdd tdd merged commit 8ceb919 into reactjs:master Mar 14, 2019
@LucasPerso LucasPerso deleted the uncontrolled-components branch March 15, 2019 09:18
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.

4 participants