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

Translate Higher Order Components #167

Merged
merged 11 commits into from
Feb 22, 2019
Merged

Conversation

agoldis
Copy link
Contributor

@agoldis agoldis commented Feb 19, 2019

No description provided.

* upstream/master: (103 commits)
  Fixes
  Fix title in refs-must-have-owner.md
  Убрал остатки испанского из очень плохого примера
  fix broken audio link in community roundup 26
  updated footer to get year based on current date
  README.md: fix link
  Fixes after review
  Update content/docs/hooks-intro.md
  Update content/docs/hooks-intro.md
  Update content/docs/hooks-intro.md
  Update content/docs/hooks-intro.md
  Update content/docs/hooks-intro.md
  Update content/docs/hooks-intro.md
  Update content/docs/hooks-intro.md
  Терминология: codemod
  Update content/docs/hooks-intro.md
  Translate MarkdownPage component
  Терминология: хук
  Терминология: TLDR
  Update hooks-intro.md
  ...
@tesseralis tesseralis mentioned this pull request Feb 19, 2019
88 tasks
@lex111 lex111 added the needs review A pull request ready to be reviewed label Feb 19, 2019
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
Co-Authored-By: agoldis <agoldis@users.noreply.github.com>
@agoldis
Copy link
Contributor Author

agoldis commented Feb 19, 2019

@lfedorov огромное спасибо за исправления! 👏🏻🍻

Copy link
Member

@lex111 lex111 left a comment

Choose a reason for hiding this comment

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

В целом, неплохо! 👌 Есть ещё над чем поработать, пока нашёл то, что явно бросается в глаза.

content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
Co-Authored-By: agoldis <agoldis@users.noreply.github.com>
@another-guy another-guy self-assigned this Feb 20, 2019
Copy link
Collaborator

@another-guy another-guy left a comment

Choose a reason for hiding this comment

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

Несколько поправок не хватает для аппрува.

В целом же, читается хорошо. Спасибо за перевод 🎉 🎉

TRANSLATION.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
content/docs/higher-order-components.md Outdated Show resolved Hide resolved
@another-guy another-guy removed their assignment Feb 21, 2019
Co-Authored-By: agoldis <agoldis@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Feb 21, 2019

Deploy preview for ru-reactjs ready!

Built with commit 9a4f20a

https://deploy-preview-167--ru-reactjs.netlify.com

Co-Authored-By: agoldis <agoldis@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Feb 22, 2019

Deploy preview for ru-reactjs ready!

Built with commit 9a4f20a

https://deploy-preview-167--ru-reactjs.netlify.com

@netlify
Copy link

netlify bot commented Feb 22, 2019

Deploy preview for ru-reactjs ready!

Built with commit 6323e57

https://deploy-preview-167--ru-reactjs.netlify.com

Copy link
Collaborator

@another-guy another-guy left a comment

Choose a reason for hiding this comment

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

@agoldis если не трудно, поправь на отображаемОЕ имя, как посоветовал Алексей. Этот вариант самый лучший.


Аппрув 🎉 🎉 🎉

@agoldis
Copy link
Contributor Author

agoldis commented Feb 22, 2019

@another-guy все исправил! Спасибо за помощь!

@another-guy another-guy removed the needs review A pull request ready to be reviewed label Feb 22, 2019
@another-guy another-guy merged commit f5d865f into reactjs:master Feb 22, 2019
@another-guy another-guy added the needs proofread A read-through is still needed to release the document label Feb 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs proofread A read-through is still needed to release the document
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants