Translate the Using Effect Hook Page #60
Conversation
|
Deploy preview for fr-reactjs ready! Built with commit 38486c7 |
JeremiePat
added
the
Pending Review
|
J'avais bien vu le problème des apostrophes : j'ai ‘ et ’ sur mon clavier, j'avais parié sur la première, mais c'était bien la deuxième ! |
tdd
added
Reviewing
There was a problem hiding this comment.
Salut @pascalvaccaro !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.
C'était un super gros boulot (j'avais pas remarqué combien cette page était longue…), tu t'en es plutôt très bien sorti, mais y'a évidemment plein de retours ; outre les petits détails et améliorations mineures, les principales sources de suggestions sont :
- « souscrire » et consorts, qui n'ont pas ce sens-là et sont avantageusement remplacées par « abonnement », « s'abonner » et « se désabonner »
- Le recours protéiforme à « contexte », qui a pourtant un sens précis en JS (qui n'est jamais celui en question), au lieu notamment de « portée » (locale ou englobante)
- « retourner » (nope!) au lieu de « renvoyer » (yep!)
- Le choix d'un ton quasi professoral et de formules allongées, là où le contexte de la page (didactique et tutoriel) nécessite des phrases plus légères et un ton moins formel. On insiste là-dessus dans les Meilleures Pratiques sur le Wiki, mais c'est parfois difficile selon le background des traducteurs…
Ne te laisse pas dépiter par le volume de retours, il est parfaitement normal que pour ta première trad, surtout sur un texte aussi long, il y en ait autant ! On sent d'ailleurs une progression au sein même de la page, ça augure donc bien pour tes futurs travaux ! 😁
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
added
Reviewed
and removed
Reviewing
|
@pascalvaccaro j'ai vraiment besoin que tu traites ces retours rapidement, c'est possible de ton côté ? |
|
Salut @tdd
Je suis en vacances jusqu'au 18 mars je ne pourrai donc pas traiter les
retours avant cette date.
J'avais mal estimé le temps des retours avant de partir, j'en suis bien
navré.
Si tu veux les traiter à ma place aucun souci sinon attendre une petite
semaine.
Pascal
…On Fri, Mar 8, 2019, 09:37 Christophe Porteneuve ***@***.***> wrote:
@pascalvaccaro <https://github.com/pascalvaccaro> j'ai vraiment besoin
que tu traites ces retours rapidement, c'est possible de ton côté ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#60 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AM-Pc_r7t1HBDU9Kg3rXfBg-rdtPDBV7ks5vUmeSgaJpZM4bJW84>
.
|
|
@pascalvaccaro aucun souci pour valider tout ça moi-même, mais ce serait pas mal qu'à ton retour tu fasse une repasse sur la PR fermée pour vérifier que tu ne t'opposais à rien… |
|
@pascalvaccaro en revanche tu n'as pas donné les droits aux mainteneur sur ta branche de PR, de sorte que je ne peux pas appliquer les changements cash dessus, je serais obligé de la merger puis de tout appliquer après coup, pénible. Peux-tu au moins changer le réglage qui va bien dans ta PR ? Y'a une case “Allow edits from maintainers”, qui ne doit pas être cochée (si elle l'est, décoche, valide, recoche, valide). |
|
@tdd je ne trouve pas la coche je me demande si la version web mobile de github ne l'aurait pas retirée... |
|
Il te suffit de demander à ton navigateur mobile d'afficher la vue desktop (dans son menu principal pour la page), tu auras le truc (faudra zoomer pour cliquer :-))
…On Fri, Mar 8, 2019, at 16:16, Pascal Vaccaro wrote:
@tdd <https://github.com/tdd> je ne trouve pas la coche je me demande si la version web mobile de github ne l'aurait pas retirée...
Je vois bien où c'est (vers unsubscribe sur la droite) mais impossible de mettre le doigt dessus sur le téléphone.
J'ai aussi regardé sur une app mobile github mais rien non plus...
Si tu as une info supplementaire je veux bien reessayer sinon il faudra suppr ma branche et en faire une nouvelle :-(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#60 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAOqNu7HoFJaGEW5zEBqGumaU8nrta0ks5vUn7igaJpZM4bJW84>.
--
Christophe Porteneuve
tdd@tddsworld.com
|
|
Merci beaucoup @tdd ! Je viens de cocher / decocher / recocher est-ce que ça fonctionne comme tu veux ? |
|
Rien à faire, ça veut pas me laisser y aller… Je comprends pas… On parle bien de la bonne PR ? |
|
Je viens de refaire la manip, je vois pas ce que je peux faire de plus... Il s'agit de ma seule PR ouverte sur ce dépôt donc peu probable qu'on se trompe :-) |
|
Peux-tu vérifier les points suivants dans les Settings de ton fork :
1. Dans *Options*, *Restrict editing to users in teams with…* est décoché (à tout hasard)
2. Dans *Branches*, y'a pas de *Branch protection rules* particulières
3. Dans *Interaction limits*, rien ne semble actif
Ce sont les seuls points où je me dis que ça peut jouer, si la case *allow edits from maintainers* est activée sur ta PR…
…On Sat, Mar 9, 2019, at 00:04, Pascal Vaccaro wrote:
Je viens de refaire la manip, je vois pas ce que je peux faire de plus...
Il s'agit de ma seule PR ouverte sur ce dépôt donc peu probable qu'on se trompe :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#60 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAOqNSPZHxOxRCkcT70JRN_QVWirK0Pks5vUuyEgaJpZM4bJW84>.
--
Christophe Porteneuve
tdd@tddsworld.com
|
|
@pascalvaccaro je sais pas si tu as pu faire les vérifs dont je te parlais hier, mais en tout cas je n'ai toujours pas les droits 😒 — je vais attendre ton retour pour que tu puisses traiter les retours toi-même : c'est quand exactement ? |
|
Salut je viens de decocher un restrict edits to collaborators dans les settings de mon fork. |
|
Putain rien à faire. Je comprends pas ce que fout GitHub :-(
On va donc se prendre 9 jours dans la vue sur ce coup :-/
…On Sun, Mar 10, 2019, at 14:47, Pascal Vaccaro wrote:
Salut je viens de decocher un restrict edits to collaborators dans les settings de mon fork.
Si ça ne fonctionne toujours pas, je rentre le 19 mars matin en France avec accès à mon pc :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#60 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAOqN8D7b89rBCIrtpPK3JmMToGlhSKks5vVQzwgaJpZM4bJW84>.
--
Christophe Porteneuve
tdd@tddsworld.com
|
|
Tu peux pas simplement copier mon fichier sur ta remote locale et tirer une nouvelle branche pour refaire la PR ? |
|
Hello,
On Sun, Mar 10, 2019, at 22:02, Pascal Vaccaro wrote:
Tu peux pas simplement copier mon fichier sur ta remote locale et tirer une nouvelle branche pour refaire la PR ?
Désolé que ce soit si galère je pensais vraiment qu'une fois la trad terminée je n'aurais pas de travail en plus :-/
« La trad terminée », c'est quand tu as traité mes retours :p
Je peux tout à fait tirer une branche sur un dépôt à partir de ta PR et faire une nouvelle PR avec (je fais automatiquement une branche locale comme ça pour mes *final tweaks* après traitement de mes revues, mais je n'en fais normalement pas une nouvelle PR), sauf que je vais devoir retranscrire toutes mes modifs à la main au lieu de les appliquer de façon fiable avec l'UI de GitHub… C'est ça qui me saoule un peu, mais je finirai par le faire si on a bouclé tout le reste avant le 19 (ce qui est peu probable, et donc l'enjeu est faible, ça ne devrait pas à soi seul retarder notre “2x100%“).
J'aimerais quand même bien comprendre quel !@# couac de config m'empêche de committer sur ta branche.
…--
Christophe Porteneuve
tdd@tddsworld.com
|
|
@pascalvaccaro la page VO a pas mal bougé, il faudra donc arbitrer les conflits. C'est un peu périlleux et je préfère m'en occuper, mais soit j'obtiens les droits sur ta branche de fork, soit je devrai tirer une branche à moi une fois que tu auras traité mes suggestions existantes. Je t'invite à faire une dernière vérif sur les protections éventuelles de branche dans ton fork, outre les protections générales. Sinon franchement aucune idée de pourquoi je me fais jeter… |
|
@pascalvaccaro autre option : tu m'ajoutes en collaborateur sur ton fork 😉 |
|
@tdd c'est fait je t'ai ajouté sur mon fork j'espère que ça va marcher comme tu veux ? |
|
@pascal sûr que oui ! Ils ont dû caler des tas de règles de protection de branche, etc. qui m'empêchaient de venir. C'est désormais OK 🎉 |
Translate the Using Effect Hook Page.
Un léger doute sur le caractère utilisé pour l'apostrophe (‘), mais rien qu'un find-n-replace ne saurait résoudre rapidement si je me suis trompé :-)
Cheers,
Pascal