Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Translation of the “Updating Objects in State” page #495

Merged
merged 6 commits into from
Jul 6, 2023

Conversation

LaureRC
Copy link
Contributor

@LaureRC LaureRC commented Jun 8, 2023

Traduction française de la page Updating Objects in State.

@github-actions
Copy link

github-actions bot commented Jun 8, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@tdd tdd added Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR scope:required-pages Part of the required translations before we can ship translator:seasoned The author of that translation is a seasoned React docs translator labels Jun 9, 2023
@tdd tdd self-assigned this Jun 9, 2023
@tdd
Copy link
Collaborator

tdd commented Jun 20, 2023

Coucou @LaureRC tu as fini le taf ou pas ? C'est encore en Draft…

@tdd tdd removed the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Jun 20, 2023
@tdd
Copy link
Collaborator

tdd commented Jul 1, 2023

@LaureRC nouveau ping : tu as fini ou pas ? Si oui marque-la "Ready for review" en bas de discussion stp

@LaureRC
Copy link
Contributor Author

LaureRC commented Jul 2, 2023

Bonjour @tdd ! Désolée pour le long délai de réponse, malheureusement j'ai été très occupée (lorsque je me suis proposée tout était bon, mais le travail et le perso m'ont très vite rattrapée !). Je me charge de mettre ça en review tout de suite !

@LaureRC LaureRC marked this pull request as ready for review July 2, 2023 16:55
@tdd tdd added the Reviewing Un mainteneur qualifié est en train de composer la revue de code label Jul 4, 2023
Copy link
Collaborator

@tdd tdd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salut @LaureRC !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.

Comme toujours, très bon boulot. On sent un recalage de certaines habitudes à mi-chemin, entre ta première passe et la dernière je suppose.

Quelques tweaks de formulation notamment, et quelques modifs que je n'ai pas reportées sur toutes les occurrences (mais je t'ai demandé de le faire dans mes commentaires).

Avant de traiter les retours, lis bien TOUT ce message, notamment les consignes d'intégration optimale des retours par lot(s)

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. J'y recalerai si besoin les usual suspects : espaces insécables avant les ponctuations doubles, et apostrophes françaises.

Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans main !

src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
- Comment mettre à jour correctement un objet dans l’état de React
- Comment mettre à jour un objet imbriqué sans le modifier
- Qu’est-ce que l’immutabilité, et comment la préserver
- Comment rendre la copie d’objet moins répétitive avec Immer
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Au passage, souviens-toi qu'hors du code, on n'a pas besoin de mettre à la main les apostrophes typographiques, finalement. Le rendu Markdown s'en occupera.

src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
src/content/learn/updating-objects-in-state.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@tdd tdd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salut @LaureRC !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.

Comme toujours, très bon boulot. On sent un recalage de certaines habitudes à mi-chemin, entre ta première passe et la dernière je suppose.

Quelques tweaks de formulation notamment, et quelques modifs que je n'ai pas reportées sur toutes les occurrences (mais je t'ai demandé de le faire dans mes commentaires).

Avant de traiter les retours, lis bien TOUT ce message, notamment les consignes d'intégration optimale des retours par lot(s)

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. J'y recalerai si besoin les usual suspects : espaces insécables avant les ponctuations doubles, et apostrophes françaises.

Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans main !

@LaureRC
Copy link
Contributor Author

LaureRC commented Jul 5, 2023

Hello @tdd ! Merci pour cette review très détaillée, je n'avais pas de retours particuliers dessus donc j'ai intégré tous tes retours en un seul commit comme demandé puis j'ai ajouté un dernier commit pour gérer les dernières occurrences de soucis que tu avais mentionné dans ta review!

Merci pour les retours positifs également, malgré le très long délai pour passer en "ready" (encore désolée pour ça) je m'étais quand même bien appliquée à suivre les consignes des bonnes pratiques et du glossaire (même si certaines ont quand même réussi à m'échapper!). Par contre je me suis faite avoir à penser devoir mettre partout les bonnes apostrophes typographiques, je saurais pour la prochaine fois que ça n'est nécessaire que dans le code! Il faudrait peut-être adapter le paragraphe concernant les apostrophes dans le document des bonnes pratiques du coup pour le préciser.

Je n'ai d'ailleurs pas traité les apostrophes qui ont été modifiées dans tes retours, ce n'est pas tout à fait homogène donc, dis moi si tu veux que je refasse une passe là-dessus, mais j'imagine que tu t'en occuperas pendant ta passe finale? Autrement tout devrait donc être bon! Fais moi signe si j'ai oublié quelque chose, je m'assurerai de réagir rapidement! 🙂

@tdd tdd force-pushed the updating-objects-in-state branch from 0adcb6e to 7fa8919 Compare July 6, 2023 08:21
@tdd
Copy link
Collaborator

tdd commented Jul 6, 2023

Dis-moi @LaureRC souhaites-tu + peux-tu tout prochainement décliner ce taf pour "Updating Arrays in State", ou est-ce préférable que je m'en occupe ?

@LaureRC
Copy link
Contributor Author

LaureRC commented Jul 6, 2023

Hey @tdd, pour être honnête je comptais initialement me proposer pour cette page, mais je ne serai pas en mesure d'avancer là-dessus avant le 20 juillet donc je comptais revenir à cette date voir si cette page avait toujours besoin d'une traduction pour me proposer. Alors si tu en as besoin rapidement je pense qu'il vaut mieux que tu t'en charges, mais si ça peut attendre que j'ai du temps disponible et que je me mette sur le sujet le 20 juillet (et je serai cette fois bien plus rapide à tout envoyer en review), je le ferai avec plaisir!

@tdd
Copy link
Collaborator

tdd commented Jul 6, 2023

Salut @LaureRC !

Écoute c'est pas urgent (je priorise les parties antérieures de Learn pour le moment), et du 21/07 au 15/08 je serai en congés et donc moins réactif, mais ça me semble être logique que tu t'en occupes si ça te va, vu que tu peux capitaliser sur ton travail pour la "version Objets". Je te l'affecte, du coup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Reviewed scope:required-pages Part of the required translations before we can ship translator:seasoned The author of that translation is a seasoned React docs translator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants