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

translated react-without-jsx #30

Merged
merged 13 commits into from
Feb 25, 2019
Merged

translated react-without-jsx #30

merged 13 commits into from
Feb 25, 2019

Conversation

jckr
Copy link
Contributor

@jckr jckr commented Feb 13, 2019

No description provided.


The component can either be provided as a string, or as a subclass of `React.Component`, or a plain function for stateless components.
Le composant peut soit être une chaîne de caractères, soit une sous-classe de `React.Component`, soit une fonction simple pour les componsants sans état.
Copy link
Contributor Author

@jckr jckr Feb 13, 2019

Choose a reason for hiding this comment

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

(todo) componsants / composants

@tdd tdd self-requested a review February 13, 2019 21:26
@tdd tdd self-assigned this Feb 13, 2019
@tdd tdd added the Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR label Feb 13, 2019
@tdd
Copy link
Collaborator

tdd commented Feb 13, 2019

@jckr je regarde ça demain, mais sois sympa, la prochaine fois signale ta « prise de main » sur une page dans #1, pour être sûrs d'éviter un doublonnement d'effort :-)

Et du coup, les autres consignes :

Important : avant de te lancer dans la traduction, je te recommande *très fortement la lecture de ces deux documents :

Par ailleurs, la lecture des revues de code pour les PR déjà traitées est très instructive aussi…

Enfin, la VF de Overreacted.io, le blog tech de Dan Abramov, est une bonne source d'inspiration en cas de doute. Et pour le reste, si tu as des interrogations, n'hésite pas à nous solliciter !

Merci pour la trad en tout cas, je relis ça vite !

@jckr
Copy link
Contributor Author

jckr commented Feb 14, 2019

merci. Mais - j'ai commenté #1 (et vérifié que personne ne s'était proposé pour cette page) avant de me lancer!

Copy link
Contributor

@gkueny gkueny left a comment

Choose a reason for hiding this comment

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

Hello 👋🏻

J'ai laissé quelques suggestions de modifications, essentiellement sur les espaces insécables et des typos. En espérant que ça aide !

content/docs/react-without-jsx.md Outdated Show resolved Hide resolved
content/docs/react-without-jsx.md Outdated Show resolved Hide resolved
content/docs/react-without-jsx.md Outdated Show resolved Hide resolved
gkueny and others added 3 commits February 14, 2019 09:46
Co-Authored-By: jckr <jerome@uber.com>
Co-Authored-By: jckr <jerome@uber.com>
Co-Authored-By: jckr <jerome@uber.com>
@jckr
Copy link
Contributor Author

jckr commented Feb 14, 2019

merci @gkueny!

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

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main. C'est globalement bien propre, j'ai juste quelques retouches à faire.

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é.

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 !

content/docs/react-without-jsx.md Outdated Show resolved Hide resolved
content/docs/react-without-jsx.md Outdated Show resolved Hide resolved
content/docs/react-without-jsx.md Outdated Show resolved Hide resolved
content/docs/react-without-jsx.md Outdated Show resolved Hide resolved
content/docs/react-without-jsx.md Outdated Show resolved Hide resolved
content/docs/react-without-jsx.md Outdated Show resolved Hide resolved
content/docs/react-without-jsx.md Outdated Show resolved Hide resolved
@tdd tdd added Reviewed and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Feb 15, 2019
tdd and others added 7 commits February 15, 2019 17:14
Co-Authored-By: jckr <jerome@uber.com>
Co-Authored-By: jckr <jerome@uber.com>
Co-Authored-By: jckr <jerome@uber.com>
Co-Authored-By: jckr <jerome@uber.com>
Co-Authored-By: jckr <jerome@uber.com>
Co-Authored-By: jckr <jerome@uber.com>
Co-Authored-By: jckr <jerome@uber.com>
@jckr
Copy link
Contributor Author

jckr commented Feb 16, 2019

Merci @tdd. Je n'ai plus l'habitude de rédiger en français, donc tes commentaires me sont très utiles! je n'ai plus le réflexe de rajouter une espace avant les : etc. En revanche, je suis un très vieil utilisateur de React, car j'ai pu l'utiliser dès 2013 chez Facebook! c'est pourquoi ça me tient à coeur de contribuer.
Bon weekend

On avait oublié de traduire les littéraux du code, tsk tsk
@tdd tdd merged commit 2f28d5d into reactjs:master Feb 25, 2019
@netlify
Copy link

netlify bot commented Feb 25, 2019

Deploy preview for fr-reactjs ready!

Built with commit 8203a23

https://deploy-preview-30--fr-reactjs.netlify.com

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