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 “Hooks FAQ” page #92

Merged
merged 6 commits into from Mar 30, 2019
Merged

Translation of “Hooks FAQ” page #92

merged 6 commits into from Mar 30, 2019

Conversation

flightcom
Copy link
Contributor

Bonsoir, voici ma contribution avec la page Hooks FAQ.
J'attends vos retours :)

@flightcom flightcom changed the title Translation of the “Hooks FAQ” Translation of “Hooks FAQ” Mar 7, 2019
@flightcom flightcom changed the title Translation of “Hooks FAQ” Translation of “Hooks FAQ” page Mar 7, 2019
@netlify
Copy link

netlify bot commented Mar 7, 2019

Deploy preview for fr-reactjs ready!

Built with commit ef56877

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

@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Mar 7, 2019
@netlify
Copy link

netlify bot commented Mar 7, 2019

Deploy preview for fr-reactjs ready!

Built with commit 2ad3f34

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

@tdd tdd self-assigned this Mar 8, 2019
@tdd
Copy link
Collaborator

tdd commented Mar 18, 2019

@flightcom j'attaque enfin la revue de ton texte, qui est long et va donc me prendre un peu de temps.

⚠️ IMPORTANT ⚠️

Le texte sous-jacent (VO) a bougé entre-temps sur l'upstream, d'où la présence de conflits dans ta PR. Aucun souci, ne te prends pas le chou, je gérerai les conflits avec une trad adaptée une fois que j'aurai fourni mes retours et que tu les auras traités, lors de ma passe locale visuelle finale avant la fusion.

Assure-toi de ton côté, dans la sidebar de ta PR, que les mainteneurs ont bien le droit de la modifier, et que ton fork ne protège pas inutilement ses branches ou limite les droits autrement, pour que ma repasse finale puisse fonctionner, notamment.

Si je t'envoi mes retours d'ici mercredi soir, auras-tu le temps de les traiter d'ici vendredi ?

Bien à toi,

@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 18, 2019
@flightcom
Copy link
Contributor Author

@tdd pas de soucis pour traiter ça d'ici vendredi, je prends l'avion pour Montréal jeudi donc j'aurai le temps de traiter ça pendant le trajet. D'ailleurs s'il reste d'autres trads à faire (plus courte que celle-ci si possible), je suis dispo :)

Concernant les permissions sur la PR, la case est bien cochée donc je pense que c'est bon, mais fais moi signe si ce n'est pas le cas.

@flightcom
Copy link
Contributor Author

Salut @tdd, as-tu pu avancer sur la revue de ma PR ? Je vais avoir un peu de temps demain soir pour faire les corrections (sachant que je suis GMT-4) sinon ce sera pour dimanche.

@tdd
Copy link
Collaborator

tdd commented Mar 22, 2019 via email

@tdd
Copy link
Collaborator

tdd commented Mar 22, 2019

Bon @flightcom c'est top, GitHub plante quand j'envoie ma PR en demandant les changements. Faut dire qu'il y en a 142 (mais t'inquiète 😉) mais bon, c'est GitHub bordel ! Ça devrait passer tout seul (et les 142 sont déjà chez eux, en prime). Je tente un ping sur Twitter pour trouver de l'aide chez eux…

@flightcom
Copy link
Contributor Author

Salut @tdd, merci pour le retour. Je vais essayer de pas m'inquiéter 😅 J'espère que ça va se débloquer rapidement...

@tdd tdd added Reviewed and removed Reviewing Un mainteneur qualifié est en train de composer la revue de code labels Mar 22, 2019
@tdd
Copy link
Collaborator

tdd commented Mar 26, 2019

@flightcom si GitHub arrive pas à débloquer leur bug, je vais de voir me fader à la main les 162 correctifs sur une copie locale de la branche… Je suis navré de pas pouvoir te les montrer avant, c'est justement là que GH déconne en refusant de finaliser la revue. Si on devait en arriver là, tu m'y autoriserais ? C'est un peu le forcing…

@flightcom
Copy link
Contributor Author

@tdd bien sûr ! Sens toi libre de corriger ce qui ne va pas :)
Je ne pense pas, mais est-ce que tu as besoin que je te donne certains accès via Github ?

@tdd tdd merged commit 862d7c1 into reactjs:master Mar 30, 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