Skip to content

Translation of Render Props #106

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
Apr 22, 2019

Conversation

vburlai
Copy link
Contributor

@vburlai vburlai commented Apr 19, 2019

Translation of Render Props

@netlify
Copy link

netlify bot commented Apr 19, 2019

Deploy preview for uk-reactjs ready!

Built with commit a83e3d6

https://deploy-preview-106--uk-reactjs.netlify.com

@oleksii-polovyi oleksii-polovyi added the Просунуті Теми Advanced Guides part of documentation label Apr 19, 2019
Copy link
Member

@oleksii-polovyi oleksii-polovyi left a comment

Choose a reason for hiding this comment

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

Great start 🚀
Just a few changes from me.
@klymenkoo @mihilbabin please review.

Copy link
Member

@vldmrkl vldmrkl left a comment

Choose a reason for hiding this comment

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

Гарна робота 👍

vburlai added 2 commits April 20, 2019 10:58
Co-Authored-By: vburlai <vitaly.burlai@gmail.com>
@vburlai
Copy link
Contributor Author

vburlai commented Apr 20, 2019

All suggestions were applied.

I did not know and applied it in one batch, but will use separate batches per person next time (to keep reviewer's name in the commit). I'm not used to this GitHub feature yet.

@vburlai
Copy link
Contributor Author

vburlai commented Apr 21, 2019

@mihilbabin changes applied. Please also see my comment regarding TRANSLATION.md.

@vburlai
Copy link
Contributor Author

vburlai commented Apr 22, 2019

Thank you guys for review.

Hi @ApolllonDev
All suggestions were applied, is it ready to be merged?

@oleksii-polovyi oleksii-polovyi merged commit 3d054c0 into reactjs:master Apr 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium Просунуті Теми Advanced Guides part of documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants