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

Translation of the "Test Utilities" page #31

Merged
merged 10 commits into from
Feb 28, 2019
Merged

Translation of the "Test Utilities" page #31

merged 10 commits into from
Feb 28, 2019

Conversation

linsolas
Copy link
Contributor

Hello all,

As I proposed, here is the french translation of the Test Utilities page.

@linsolas
Copy link
Contributor Author

By the way, in the official doc, there is a broken link (a wrong anchor to be correct) in the "Note" block:

We recommend using shallow rendering

which points to https://reactjs.org/docs/test-utils.html#shallow-rendering

I didn't correct it in my translated page, but I suspect that it should point to https://reactjs.org/docs/shallow-renderer.html.

I will do a PR on the official doc I think...

@linsolas
Copy link
Contributor Author

The PR has been done: reactjs/react.dev#1684
If it is merged, I will update my translation...

@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 14, 2019
Copy link
Contributor

@gkueny gkueny left a comment

Choose a reason for hiding this comment

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

Hello 👋🏻

I left some changes suggestions. I hope this helps

gkueny and others added 3 commits February 14, 2019 10:07
Co-Authored-By: linsolas <linsolas@gmail.com>
Co-Authored-By: linsolas <linsolas@gmail.com>
Co-Authored-By: linsolas <linsolas@gmail.com>
@linsolas
Copy link
Contributor Author

Hello @gkueny,

Thanks for this first review!

@tdd tdd self-assigned this Feb 14, 2019
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.

Merci pour ta traduction :) Je me suis permis de faire des suggestions de style pour que ça sonne un peu plus français. Je laisse @tdd faire une passe supplémentairement, il y a quelque point ou ce sera bien d'avoir son avis.

>
> Alternatively, Airbnb has released a testing utility called [Enzyme](http://airbnb.io/enzyme/), which makes it easy to assert, manipulate, and traverse your React Components' output.
> Autrement, Airbnb a publié un utilitaire de test appelé [Enzyme](http://airbnb.io/enzyme/), qui rend facile les assertions, la manipulation et la traversée du rendu de vos composants React.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
> Autrement, Airbnb a publié un utilitaire de test appelé [Enzyme](http://airbnb.io/enzyme/), qui rend facile les assertions, la manipulation et la traversée du rendu de vos composants React.
> A titre d'alternative, Airbnb a publié un utilitaire de test appelé [Enzyme](http://airbnb.io/enzyme/) qui permet facilement de tester des assertions, de manipuler vos composants React et d'en explorer le rendu.

Copy link
Collaborator

Choose a reason for hiding this comment

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

J'ai fait beaucoup de littérature ici pour que ça sonne plus "Français". C'est très subjectif.

@tdd Je ne suis pas bien sur de comment il faut traduire "traverse your React Components' output." Traverser n'est pas très français et je suggère explorer mais j'ai un gros, gros doute.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Personnellement, je préfère en effet ta tournure de phrase. Quant à ton interrogation, je pense que l'on garde quand même l'idée générale.

@@ -296,7 +296,7 @@ Same as [`scryRenderedComponentsWithType()`](#scryrenderedcomponentswithtype) bu
renderIntoDocument(element)
```

Render a React element into a detached DOM node in the document. **This function requires a DOM.** It is effectively equivalent to:
Réalise le rendu d'un élément React au sein d'un nœud du DOM détaché dans le document. **Cette fonction nécessite un DOM**. C'est effectivement équivalent à :
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Réalise le rendu d'un élément React au sein d'un nœud du DOM détaché dans le document. **Cette fonction nécessite un DOM**. C'est effectivement équivalent à :
Réalise le rendu d'un élément React au sein d'un nœud du DOM détaché du document. **Cette fonction nécessite un DOM**. C'est en fait équivalent à :

Copy link
Collaborator

Choose a reason for hiding this comment

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

@tdd je veux bien ton avis ici. La notion de nœud DOM détaché est confuse et même en anglais je ne trouve pas ça très claire ici (le nœud est détaché de l'arbre DOM du document mais il fait toujours parti du document)

@JeremiePat JeremiePat added Reviewed and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Feb 22, 2019
@linsolas
Copy link
Contributor Author

Bonjour @JeremiePat

Merci pour ta revue, je commence la correction.

@netlify
Copy link

netlify bot commented Feb 24, 2019

@linsolas
Copy link
Contributor Author

Merci @JeremiePat pour ta revue, je pense avoir intégré tes remarques. A voir les retours de @tdd maintenant :)

@tdd tdd added Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Feb 27, 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 @linsolas !

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

Bon, pas mal de petits retours comme toujours, de l'harmonisation avec les autres pages d'API à des détails de singulier/pluriel, en passant par quelques ajustements sémantiques. Mais ça reste super 😉

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, 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 !

@tdd tdd removed the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 28, 2019
Co-Authored-By: linsolas <linsolas@gmail.com>
@linsolas
Copy link
Contributor Author

Bonjour @tdd,

Sauf erreur de ma part, j'ai intégré toutes tes remarques. Ca améliorera mes prochaines traductions aussi :)

@tdd
Copy link
Collaborator

tdd commented Feb 28, 2019

@linsolas j'ai vu, j'attends juste les checks par acquis de conscience avant de faire ma repasse visuelle / insécables finale en local en prévision du merge 😉

@tdd tdd merged commit 7bb6514 into reactjs:master Feb 28, 2019
@linsolas linsolas deleted the test-utilities branch February 28, 2019 15:31
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

4 participants