Skip to content

Conversation

@nina-alin
Copy link
Contributor

Un des composants de react-dom que j'adore utiliser 🛩️

@github-actions
Copy link

github-actions bot commented Aug 25, 2023

Size changes

Details

📦 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 self-assigned this Aug 28, 2023
@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:beginner The author of this translation has little to no prior doc translation experience labels Aug 28, 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 @nina-alin !

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

L'essentiel des tweaks vient de "portal" -> "portail" (à toi de propager les tweaks manquants, je ne les ai faits que si je touchais à autre chose) et à "en-dehors" (y'a un tiret).

Quelques glissements sémantiques et éclaircissements, mais sinon super 👍

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 !

@nina-alin
Copy link
Contributor Author

nina-alin commented Aug 29, 2023

Hello @tdd ! J'ai bien commit tes suggestions et j'ai replacé portail partout ! Merci encore pour le temps que tu prends pour nous former à la traduction !

@tdd
Copy link
Collaborator

tdd commented Aug 29, 2023

Coucou @nina-alin ! OK super je fais une dernière repasse visuelle, un rebase sur le main à jour et en avant.

Merci à toi ! 🙏

@tdd tdd force-pushed the feat/translation-of-create-portal branch from ebe5c49 to df93073 Compare August 29, 2023 09:11
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:beginner The author of this translation has little to no prior doc translation experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants