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

Error boundaries #67

Merged
merged 8 commits into from Mar 9, 2019
Merged

Error boundaries #67

merged 8 commits into from Mar 9, 2019

Conversation

linsolas
Copy link
Contributor

Bonjour,

Voici ma proposition de traduction pour la page Error Boundaries.

La traduction de Error boundaries n'est effectivement pas simple, j'ai mis Limiteur d'erreur mais je ne suis pas vraiment satisfait.
Soit on trouve mieux, soit on laisse, soit on garde le terme anglais (en mettant une petite note de traduction au début).

/poke @tdd :)

@linsolas
Copy link
Contributor Author

un point aussi : j'ai toujours du mal à savoir si je dois mettre au pluriel après le "de" (gestionnaire d'événement / limiteur d'erreur). Du coup en général j'accorde tout ou rien (=> "un limiteur d'erreur", "des limiteurs d'erreurs"). ..

@netlify
Copy link

netlify bot commented Feb 25, 2019

Deploy preview for fr-reactjs ready!

Built with commit b4b6d05

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

@tdd
Copy link
Collaborator

tdd commented Feb 25, 2019 via email

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

Oups 😓 Pourtant je consulte assez régulièrement le glossaire, mais je n'y ai pas pensé pour Error Boundaries...
Je vais mettre à jour mes 2 PR !

@linsolas
Copy link
Contributor Author

PR mises à jour

@JeremiePat JeremiePat added the Reviewing Un mainteneur qualifié est en train de composer la revue de code label Mar 5, 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.

Salut :)

Quelque suggestion de reformulation pour sonnez un peu plus français dans certain cas plus quelque bricoles mais globalement c'est nickel :) (Note: tu peux directement accepter les suggestions plutôt que de créer les commits toi même)

content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
@JeremiePat JeremiePat added Reviewed and removed Reviewing Un mainteneur qualifié est en train de composer la revue de code labels Mar 5, 2019
Co-Authored-By: linsolas <linsolas@gmail.com>
Copy link
Contributor Author

@linsolas linsolas left a comment

Choose a reason for hiding this comment

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

Merci @JeremiePat pour ta relecture. J’ai noté 4 propositions avec des petites erreurs. Te serait-il possible de les mettre à jour, que je puisse les intégrer plus facilement (je suis en vacances et je fais ça sur mon téléphone ce qui n’est guère pratique ;) ) ?

content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
@JeremiePat
Copy link
Collaborator

Désolé, je suis allé un peu vite :-/ Tout est corrigé, tu peux commiter les suggestions en les acceptant.

Co-Authored-By: linsolas <linsolas@gmail.com>
@linsolas
Copy link
Contributor Author

linsolas commented Mar 6, 2019

Hello @JeremiePat

C’est fait ! Merci pour ta relecture.

@tdd tdd self-assigned this Mar 8, 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, me revoilà !

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

Bon, comme toujours beaucoup de bonnes choses, même si j'ai plein de petites retouches et clarifications à suggérer, comme d'hab', car je suis un Ayatollah™, comme tu le sais 😁

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 !

content/docs/error-boundaries.md Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
@tdd tdd removed the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Mar 8, 2019
tdd and others added 2 commits March 9, 2019 00:17
Co-Authored-By: linsolas <linsolas@gmail.com>
@tdd tdd merged commit 0e8aa0e into reactjs:master Mar 9, 2019
@linsolas linsolas deleted the error-boundaries branch March 9, 2019 15:06
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