Translate "Implementation Notes" page

[forresst]

Traduction terminée, merci pour vos remarques

[netlify]

Deploy preview for fr-reactjs ready!

Built with commit 35852c3

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

[forresst]

Bonjour tout le monde,

J'ai commencé la traduction, le travail est en cours (c'est la première fois que j'utilise Draft Pull Request), j'espère que ce sera un beau bébé.

@tdd J'essaye d'en prendre bien soin, et justement j'ai deux remarques, qui ne sont pas bloquantes pour avancer mais j'avoue qu'un petit éclaircissement sur ces deux points, permettra de continuer le reste de la traduction avec plus de sérénité :

1. Nous avons une incohérence pour la traduction de « reconciler », dans le glossaire du wiki, nous avons « réconcilieur » comme traduction alors que dans la VF de Overreacted.io, c'est traduit par « réconciliateur ». Alors quelle est la bonne traduction ? Pour l'instant j'ai pris « réconciliateur » (je changerai si besoin).
2. Je suis pas satisfait sur la traduction des deux phrases aux lignes 51 et 53, une proposition serait la bienvenue

[tdd]

@forresst alors :

1. honte sur moi, ma VF chez Overreacted a raison et j'ai foiré les reviews du terme ici. Je ferai un commit transverse sur master pour régler ça une bonne fois pour toutes
2. Ligne 51, il s'agit d'exprimer que peu importe l'approche, au final le réconciliateur aura l'info : il saura quel est l'élément produit par le rendu. Je dirais donc : « Dans les deux cas, le réconciliateur saura quel élément a été produit par le rendu de App »
3. Ligne 53, je propose :

« Ce processus est récursif. Le rendu de App peut produire un <Greeting />, le rendu de Greeting peut produire un <Button />, et ainsi de suite. Le réconciliateur « s’enfoncera » de manière récursive dans les composants définis par l'utilisateur, et saura ainsi ce que produit le rendu de chacun. »

Deux remarques sur cette base :

• Tu as raison de faire ce genre de demandes tôt, pour m'éviter 100% de rephrasé à la review. Tu te doutes bien que « faire le rendu à » ne veut rien dire et n'est pas français, c'est donc voué à l'échec. Par ailleurs, si tu lis les MP, le Glossaire et surtout les PR déjà validées, tu verras qu'on essaie de fuir « rendre » et « faire le rendu », pour trouver des formes moins gauches. D'où les suggestions de rendu comme un substantif ici. Vu qu'on parle du réconciliateur et des détails d'implémentation, se contenter de l'habituelle simplification « affichage » n'est pas une option.
• Je l'ai précisé en amont, les pages de How to Contribute sont très exigeantes en termes de traduction ; il est impératif d'avoir une bonne connaissance de fond de React pour les traduire impeccablement ; je n'ai pas voulu les verrouiller en collant mon nom d'entrée de jeu dessus, mais il est important d'en sentir la difficulté quand on les attaque.

Bien à toi,

[tdd]

@forresst c'est un énorme bébé, je vais sans doute te faire deux voire trois revues séquentielles, pour que tu puisses traiter au fil de l'eau, mais surtout pour éviter de tomber dans le bug de ouf de GitHub qu'on se tape sur #92 et qui me rend fou 😁

[forresst]

C'est un énorme bébé, je vais sans doute te faire deux voire trois revues séquentielles, pour que tu puisses traiter au fil de l'eau, mais surtout pour éviter de tomber dans le bug de ouf de GitHub qu'on se tape sur #92 et qui me rend fou 😁

@tdd Pas de soucis ! Si je peux en plus t'éviter l'asile ... 😄

[tdd]

Salut @forresst !

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)

Écoute je suis super soulagé en fait, c'est du super boulot 😎 🥇… Du coup j'ai beau être à 70% du contenu je n'ai pour le moment « que » 42 retours, ce qui est très bon signe. Je te fais le reste dans l'après-midi, mais n'hésite pas à traiter ça avant si tu as le temps. En revanche évite de recaler des trucs autres sur la source, je risquerai de faire la suite de la revue en même temps et que ça se marche sur les pieds entre nos deux efforts…

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]

Salut @forresst !

Voici la 2e partie ; il en reste une dernière, mais je conserve des lots réduits (~50) de suggestions pour ne pas titiller GitHub 😒… Note que je suis repassé sur certains oublis de trad normalisée de la première partie, mais ça ne devrait pas gêner si tu les traites séquentiellement.

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]

@forresst quand tu vas traiter les retours : utilise bien l'onglet Files Changed, car :

1. Ça évitera de masquer connement les 2/3 des retours, comme le fait par défaut le fil de conversation du premier onglet
2. Ça te donnera la possibilité d'agréger les acceptations de suggestions en lots, pour éviter une pléthore de commits et de notifications…

[tdd]

Salut @forresst !

Et voilà, partie 3/3. On en voit le bout ! 😁

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 !

[forresst]

@tdd Tout d'abord merci pour ta relecture. Je t'ai pas facilité la tâche sur certains points (Désolé).

J'ai appliqué toutes tes propositions de modification à l’exception de deux remarques (j'ai commenté les deux remarques). Pour ces deux remarques, comme je sais que tu veux faire une dernière repasse, je te laisse le faire ? Où je fais un commit pour ça ?

Encore une fois merci pour ton aide

[tdd]

@forresst m'as-tu donné les droits sur ta PR ? Tu as le mode d'emploi ici. Sans ça je ne peux pas retranscrire proprement ma repasse finale sur cette PR et je serai forcé d'en tirer une autre à partir de l'état courant de celle-ci… 😒

[forresst]

@tdd Euh ! Là je comprends pourquoi tu ne peux pas, j'ai déjà ce qu'il faut depuis le premier jour de ma PR :

[tdd]

@forresst il se peut que ton dépôt ait des règles de protection de branche. Vérifie les points suivants dans ses Settings :

1. Section Options, partie Features : Restrict editing to users in team with push access doit être décoché
2. Section Branches, partie Branch protection rules : vérifie que ta branche de PR ne fait pas l'objet d'une protection spécifique
3. Si tout ça semble bon, tu peux carrément m'ajouter dans les Collaborators de ton fork, en bas de la section Collaborators & teams.

Tiens-moi au jus

[forresst]

@tdd Argh ! Section Options, partie Features : Restrict editing to users in team with push access n'était pas décoché.

J'ai donc fait le nécessaire. Et pour être sûr je t'ai rajouté comme collaborators.

Désolé je ne connaissais pas l'option Restrict editing to users in team with push access.

[tdd]

@forresst bon clairement je ne trouve aucune identité publique pour toi dans tes commits, dans Twitter, etc. tu veux juste « Forresst » dans les remerciements de trad ?

[forresst]

@tdd J'ai un compte twitter