Skip to content

Translate Lists and Keys page to French #15

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

Merged
merged 4 commits into from
Feb 13, 2019
Merged

Conversation

thib92
Copy link
Contributor

@thib92 thib92 commented Feb 11, 2019

Translated the page "Lists and Keys" to French.

@tdd
Copy link
Collaborator

tdd commented Feb 11, 2019

@thib92 c'est prêt pour une review du coup, faute de WIP ?

@tdd tdd self-assigned this Feb 11, 2019
@thib92
Copy link
Contributor Author

thib92 commented Feb 11, 2019

@tdd Oui, prêt !

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 @thib92 !

On sent déjà des progrès par rapport à la précédente, c'est cool ! 💯

Comme d'hab', pas mal de petites remarques pour viser le top, je te laisse traiter et faire tes retours.

Au fait, si tu ne l'as pas encore vu, jette un œil à cette reco de traitement des suggestions.

@tdd tdd added the Reviewed label Feb 12, 2019
@thib92
Copy link
Contributor Author

thib92 commented Feb 12, 2019

Hello @tdd ! Merci pour la review !
Je ne fais pas partie de la team donc je n'ai pas accès à la page que tu m'as donnée.

@tdd
Copy link
Collaborator

tdd commented Feb 12, 2019

@thib92 ah merde OK :-) En fait ça dit juste de regrouper les acceptations de suggestion par lots à l'aide du deuxième bouton plutôt que du "commit suggestion", pour alléger les histos et les e-mails de notifs…

@thib92
Copy link
Contributor Author

thib92 commented Feb 12, 2019

Oui je l'ai fait dans ma dernière PR (enfin je crois ?) Tu as reçu 50 e-mails ?

@tdd
Copy link
Collaborator

tdd commented Feb 12, 2019

@thib92 Hmmm toi non, mais sur d'autres cas oui :-)

@thib92
Copy link
Contributor Author

thib92 commented Feb 12, 2019

Merci @tdd pour la review ! J'ai appliqué toutes les suggestions. Je finis les derniers points et je te ping quand tout est good 😄

@Geslain
Copy link
Contributor

Geslain commented Feb 12, 2019

@tdd je me sens un peu concerné, j'avais pas vu la fonction de regroupement de commit de suite 😅

@thib92
Copy link
Contributor Author

thib92 commented Feb 13, 2019

@tdd voilà j'ai appliqué les deux commentaires sur lesquels tu n'avais pas fait de suggestion automatique 😉

@tdd
Copy link
Collaborator

tdd commented Feb 13, 2019

@thib92 au top 🙏, je fais une dernière passe de revue visuelle en local, après quoi je mergerai.

Suite à revue visuelle :

- Espaces insécables devant les signes de ponctuation doubles et dans les guillemets français
- Utilisation de la nouvelle capitalisation des titres (française), fraîchement confirmée
- Quelques retouches cosmétiques pour éviter les orphelines
@tdd
Copy link
Collaborator

tdd commented Feb 13, 2019

Salut @thib92 après revue visuelle, j'ai rajouté un petit commit de finalisation. J'ai remarqué que tu as très peu utilisé les insécables, et mis des espaces classiques à la place, penses-y la prochaine fois stp !

Je merge, merci pour tout le taf ! 🙏

@tdd tdd merged commit 3c39c9c into reactjs:master Feb 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants