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

Traduction de la référence API React.Component #22

Merged
merged 4 commits into from
Feb 15, 2019

Conversation

tdd
Copy link
Collaborator

@tdd tdd commented Feb 13, 2019

No description provided.

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

tdd commented Feb 13, 2019

@JeremiePat juste pour être sûr, tu as vu que j'ai sollicité ta review ici ? 😘

@JeremiePat
Copy link
Collaborator

Oui, oui, c'est en cours ;)

Copy link
Collaborator

@JeremiePat JeremiePat left a comment

Choose a reason for hiding this comment

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

Et voila ! Sacré gros morceau.

content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
content/docs/reference-react-component.md Outdated Show resolved Hide resolved
@JeremiePat JeremiePat added Reviewed and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Feb 13, 2019
@tdd
Copy link
Collaborator Author

tdd commented Feb 13, 2019

Hello @JeremiePat ! Merci pour la revue mec. J'ai tout mis à jour. Il reste 3-4 points où on n'est pas 100% d'accord, j'ai répondu et laissé les convos non-résolues, tu peux jeter un œil ?

Et c'est moi où tu as lâché l'affaire, hors les titres, vers 70% du contenu ? 😛

@tdd tdd force-pushed the reference-react-component branch from 205b962 to f0b96d9 Compare February 13, 2019 21:56
@tdd
Copy link
Collaborator Author

tdd commented Feb 14, 2019

@JeremiePat si tu as un moment aujourd'hui pour revenir sur les 3-4 points que j'ai contestés… 😉

@JeremiePat
Copy link
Collaborator

Désolé, j’étais totalement indisponible hier. Les deux points sur lesquels je chipote, je ne me battrait pas indéfiniment dessus. Si tu continue d'objecter, tu peux souverainement décider que bon ça va :P

@tdd
Copy link
Collaborator Author

tdd commented Feb 15, 2019

OK donc je décide souverainement 😉

@tdd tdd merged commit 248f71c into master Feb 15, 2019
@tdd tdd deleted the reference-react-component branch February 15, 2019 09:59
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.

2 participants