Skip to content

Translation of Render Props#106

Merged
oleksii-polovyi merged 4 commits intoreactjs:masterfrom
vburlai:translation-of-render-props
Apr 22, 2019
Merged

Translation of Render Props#106
oleksii-polovyi merged 4 commits intoreactjs:masterfrom
vburlai:translation-of-render-props

Conversation

@vburlai
Copy link
Copy Markdown
Contributor

@vburlai vburlai commented Apr 19, 2019

Translation of Render Props

@netlify
Copy link
Copy Markdown

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
Copy Markdown
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.

Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Copy link
Copy Markdown
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.

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

Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
vburlai added 2 commits April 20, 2019 10:58
Co-Authored-By: vburlai <vitaly.burlai@gmail.com>
@vburlai
Copy link
Copy Markdown
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.

Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
Comment thread content/docs/render-props.md Outdated
@vburlai
Copy link
Copy Markdown
Contributor Author

vburlai commented Apr 21, 2019

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

@vburlai
Copy link
Copy Markdown
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