hooks-intro translation

[emmadal]

Ceci est mon premier PR concernant la page intro du hooks

[netlify]

Deploy preview for fr-reactjs ready!

Built with commit 71544c3

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

[tdd]

@emmadal tu devrais la faire en Draft (je ne sais pas si c'est possible une fois normale, mais bon). En tous les cas je la tague [WIP], tu changeras le titre une fois qu'elle sera prête pour revue.

[emmadal]

Compris. mais la Draft je n'y comprends pas trop mais bon, @tdd. Je peux maintenant modifier le titre?

[tdd]

Salut @emmadal !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main. Bon, tu n'as pas encore fini (j'ai cru à tort que si, d'où cette revue), mais voilà déjà de quoi traiter tout ce que tu as envoyé.

Avant de traiter les retours, lis bien TOUT ce message, notamment les consignes d'intégration optimale des retours par lot(s)

Au global, je suis un peu surpris. On a vraiment le sentiment que ça a été fait par plusieurs personnes, ou alors par petits bouts discontinus lors de sessions avec un état de fatigue très variable 😉. Certains paragraphes ont une très bonne forme, d'autres alignent un mot-à-mot qui ne veut parfois pas dire grand-chose… c'est très curieux au final !

Un gros nombre de retours touchent principalement à de la syntaxe : erreurs d'accents, de nombre (singulier/pluriel) ou de genre (masculin/féminin), qui sont pléthoriques. De la forme donc, et non du fond.

On retrouve toutefois aussi pas mal de contre-sens, qui laissent penser qu'on touche pour un certain nombre de questions à des sujets que tu ne maîtrises pas (encore 😉) suffisamment sur React.

Mais bon, il faut bien commencer quelque part, pas vrai ? C'est un gros boulot, et tu t'en est sorti très honorablement, donc continue, finis-nous cette PR et sabre le champagne une fois la fusion terminée 🍾

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 !

[tdd]

@emmadal y'a au moins deux paragraphes en gras dans mon intro de revue qui te demandent de regrouper tes acceptations de suggestions via le bouton dédié, et du coup tu me fais quand même un commit par suggestion… 😢 Tu peux regrouper le reste s'il te plaît ?

[emmadal]

Merci infnimiment @tdd d'apprecier le travail et de m'encourager.
J'ai lu les corrections que vous aviez apporte a ma PR et je les appliquerait,
😄 c'est vrai que j'ai traduit textuellement de l'anglais au francais, mais je tacherais de bien traduire et terminer ma PR ce Lundi. Je serais le plus heureux de voir ma PR dans la master 😃

[tdd]

@emmadal attention aussi, en traversant la liste des suggestions tu as apparemment sauté le “24 hidden" au milieu, qu'il faut cliquer pour voir les… 24 suggestions manquantes 😉

[tdd]

@emmadal (deux fois d'ailleurs, ça charge le rab' par lots de 20)

[emmadal]

oui je viens de le voir desole. je les appliquerais

[tdd]

Mais enfin @emmadal, pas un commit par suggestion, bon sang ! Tu m'as dit que tu allais regrouper la suite ! Tu as un bouton pour ça quand tu regardes les suggestions dans l'onglet Files Changed de la PR…

[emmadal]

Je n'avas pas bien compris, jai visiter l'onglet Files changed,
Je vois deux bouttons add suggestion to batch et commit suggestion.
je ne sais pas comment les regrouper tous 😢
ou bien le boutton Review change ?? 😢

[tdd]

@emmadal hello !

Alors, tu fais un "Add suggestion to batch" à chaque acceptation. Et quand tu as fini, tout en haut de la page (et dans la barre fixe du haut) tu as un "Commit suggestions" qui envoie le tout d'un coup.

C'est un peu la même différence qu'entre un git add et un git commit, tu vois ?

[emmadal]

J'ai compris parfaitement cette fois-ci

[emmadal]

Desole @tdd pour l'imcomprehension, mais je crois que jai reussi a les appliquer en un seul commit comme vous me les avez demande

[tdd]

@emmadal vu le titre ("Apply suggestions…") oui 👏 🎉

Saurais-tu me dire d'ici combien de temps tu devrais avoir fini ta traduction ? Il restait quelques parties…

[emmadal]

je finirais avant 15h, heure locale de la Cote d'ivoire mon pays

[emmadal]

j'etais tres occupé pendant ces temps-ci raison pour laquelle je ne travaillais pas sur la traduction. je la terminerais aujourd'hui

[emmadal]

j'ai fini la page de traduction @tdd

[tdd]

@emmadal ouah, je ne sais pas ce que tu as fichu avec tes branches et ton fork, mais on y retrouve désormais la moitié de l'historique du dépôt, et pas du tout que ton propre taf.

C'est très, très gênant pour la review.

Il semble que ça soit ton tout dernier commit, qui remerge le master de référence chez toi, qui soit à l'origine du problème. Il ne rajoute aucun travail de trad à toi sur le truc, il faudrait que tu le vires.

Pour faire ça en local, voici comment t'y prendre, en supposant que tu es sur ta branche :

git reset --hard HEAD@{1}
git push --force

Bien à toi,

[emmadal]

J'ai effectuee les instructions que vous m'aviez donne, je crois que cette fois-ci c'est la bonnne

[tdd]

Salut @emmadal !

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)

Bon, j'ai revu les quelques parties qui te restaient à traduire. On sent le côté « pressé par le temps », y'avait un certain nombre de contre-sens. À part ça, ce ne sont que des petites typos et fautes d'accord, pour l'esentiel.

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 !

[emmadal]

Je les aie appliquer en un seul commit vos corrections. Votre coup de main a ete geniale 👍 👍

[tdd]

@emmadal ayé c'est dans master, donc sur fr.reactjs.org dans quelques instants 👏

[emmadal]

@emmadal ayé c'est dans master, donc sur fr.reactjs.org dans quelques instants

🍾 🍾 Super content.
Puis-je avoir d'autres pages et continuer aussi??? SVP @tdd

[tdd]

@emmadal pour continuer à progresser tranquillement, je te propose de prendre la main sur le billet de blog 2018/12/19/react-v-16-7 ? (Il est ici)

Si ça te va dis-moi, je te l'affecterai

[emmadal]

@emmadal pour continuer à progresser tranquillement, je te propose de prendre la main sur le billet de blog 2018/12/19/react-v-16-7 ? (Il est ici)

Si ça te va dis-moi, je te l'affecterai

oui il me vas @tdd je suis en attente

[emmadal]

il me va parfaitement

…

On Wed, 20 Mar 2019 at 11:37, Christophe Porteneuve < ***@***.***> wrote: @emmadal <https://github.com/emmadal> pour continuer à progresser tranquillement, je te propose de prendre la main sur le billet de blog 2018/12/19/react-v-16-7 ? (Il est ici <https://reactjs.org/blog/2018/12/19/react-v-16-7.html>) Si ça te va dis-moi, je te l'affecterai — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#99 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/Akm6GPbz4FmbscnYw0h-bhcbyUrumWzDks5vYg9ZgaJpZM4bp7-T> .

[tdd]

Done!