Ref dom #109
Ref dom #109
Conversation
update fork
sheplu
added
the
Pending Review
|
Deploy preview for fr-reactjs ready! Built with commit 450784e |
tdd
added
Reviewing
There was a problem hiding this comment.
Salut @sheplu ! Tu vas bien ?
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, pour être très franc, ce n'est pas ta meilleure trad… 😁 Des trads oubliées, du mot-à-mot très sensible, plein de termes hors des clous (Glossaire ou trads existantes sur le thème)…
Et quelques trucs qui entraînent de nombreux ajustements pour peu de raisons, notamment du « référence » au lieu de « ref » partout, « noeud » au lieu de « nœud », etc.
Ça sent un peu la fatigue ? Mais ça a, évidemment, le mérite d'exister et de faire avancer les choses. Voici en tout cas mes retours, pour pouvoir boucler cette trad rapidement et nous approcher de la ligne d'arrivé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
added
Reviewed
and removed
Reviewing
Co-Authored-By: sheplu <[email protected]>
|
@sheplu au top, je fais une dernière repasse visuelle locale et ce sera bon 👍 |
|
merci pour la review @tdd |
Références et DOM
quelques doutes sur deux ou trois traductions qui se reproduisent, ou sur certains endroit je pense qu'il y a mieux