Faq internals #70
Faq internals #70
Conversation
|
Deploy preview for fr-reactjs ready! Built with commit fad4e14 |
tdd
added
the
Pending Review
JeremiePat
added
the
Reviewing
JeremiePat
added
Reviewed
and removed
Reviewing
Co-Authored-By: lbelavoir <lbelavoir@users.noreply.github.com>
There was a problem hiding this comment.
Salut @lbelavoir !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.
Comme toujours, beaucoup de formules nickel… mais quelques glissements sémantiques 😉. Et ces damnés guillemets français…
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
removed
the
Pending Review
Co-Authored-By: lbelavoir <lbelavoir@users.noreply.github.com>
Plusieurs points de discussion avant de soumettre une PR:
Internalsa été traduit parfonctionnalitésmais je ne suis pas certaine de cette traductionla gestion des événements et la mise à jour manuelle du DOM que **sinon** vous auriez dû utiliser pour créer votre application.j'avoue ne pas vraiment être sûre de où placersinondans cette phrase.React FiberetFiberdans le dernier paragraphe ?Merci.