Translation of the API Reference "ReactDOMServer" #51
Conversation
Geslain
changed the title
Geslain
changed the title
Geslain
force-pushed
the
reference-react-dom-server
branch
from
917bc2b to
99e5ec0
Compare
|
Hello, Je me suis un peu plus "laché" sur cette traduction par rapport à la première, tout en respectant au max les meilleures pratiques. Merci d'avance ! |
JeremiePat
added
the
Pending Review
There was a problem hiding this comment.
Salut @Geslain !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.
L'essentiel des retours vise une fluidification / allègement des formules, et on trouve quelques petits tweaks récurrents (ex. « côté » qui prend un circonflexe, « Remarque » pour “Note”, etc.). Ne sois pas gêné par le volume de retours, c'est juste qu'on vise le top ! 😉
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
Pending Review
|
@Geslain ce serait vraiment cool que tu aies le temps de traiter mes retours aujourd'hui ; on essaie de boucler le Core pour ce soir… |
|
@tdd je le fais en fin de journée sans faute 😊 |
Co-Authored-By: Geslain <geslain.dahan@gmail.com>
|
@tdd C'est bon de mon coté :) |
|
Deploy preview for fr-reactjs ready! Built with commit c11133e |
|
Deploy preview for fr-reactjs ready! Built with commit d452200 |
No description provided.