Skip to content

Translate lifting-state-up.md - #238

Merged
another-guy merged 8 commits into
reactjs:masterfrom
rlax:patch-1
Mar 11, 2019
Merged

Translate lifting-state-up.md#238
another-guy merged 8 commits into
reactjs:masterfrom
rlax:patch-1

Conversation

@rlax

@rlaxrlax commented Mar 7, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@netlify

netlifyBot commented Mar 7, 2019•
edited
Loading

Copy link
Copy Markdown

Deploy preview for ru-reactjs ready!

Built with commit 7204cf7

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

@rlaxrlax mentioned this pull request Mar 7, 2019
Closed
88 tasks
@gcorgcor added the needs review A pull request ready to be reviewed label Mar 7, 2019
angryermine
angryermine reviewed Mar 7, 2019
View reviewed changes

@angryermineangryermine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Предложения по улучшению перевода

Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
@angryermine@rlax
Apply suggestions from code review
ea32146
Co-Authored-By: rlax <rlax@users.noreply.github.com>
ntishkevich
ntishkevich approved these changes Mar 9, 2019
View reviewed changes

@ntishkevichntishkevich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Перечитал несколько раз. ничего не заметил. Отличная работа 🙌 ! Нужно ревью от @lex111 и @another-guy.

lex111
lex111 reviewed Mar 10, 2019
View reviewed changes

@lex111lex111 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Предложения для улучшения перевода

Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
@lex111@rlax
Apply suggestions from code review@lex111
ded8485
Co-Authored-By: rlax <rlax@users.noreply.github.com>
@ntishkevich

ntishkevich commented Mar 10, 2019

Copy link
Copy Markdown
Contributor

Также могли бы перевести еще название этой статьи в файле nav.yml? Спасибо.

@rlax

rlax commented Mar 10, 2019

Copy link
Copy Markdown
ContributorAuthor

@ntishkevich пожалуйста ;-)

lex111and others added 3 commits March 10, 2019 15:23
@lex111@rlax
Update content/docs/lifting-state-up.md
189a9d8
Co-Authored-By: rlax <rlax@users.noreply.github.com>
@lex111@rlax
Update content/docs/lifting-state-up.md
1b18077
Co-Authored-By: rlax <rlax@users.noreply.github.com>
@rlax
Update lifting-state-up.md
bd591dc
gcor
gcor reviewed Mar 11, 2019
View reviewed changes
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
another-guy
another-guy reviewed Mar 11, 2019
View reviewed changes

@another-guyanother-guy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Небольшие поправки к переводу, который уже в очень хорошем состоянии! Спасибо, @rlax!!!

Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
Comment threadcontent/docs/lifting-state-up.md Outdated
@rlax
Apply suggestions from code review
7204cf7
Co-Authored-By: rlax <rlax@users.noreply.github.com>
@another-guy
another-guy merged commit aa5011b into reactjs:masterMar 11, 2019
@another-guyanother-guy added needs proofread A read-through is still needed to release the document and removed needs review A pull request ready to be reviewed labels Mar 11, 2019
@another-guy

another-guy commented Mar 11, 2019

Copy link
Copy Markdown
Contributor

@rlax спасибо за оперативность !

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

5 more reviewers
@angryermineangryermineangryermine left review comments
@lex111lex111lex111 left review comments
@gcorgcorgcor left review comments
@another-guyanother-guyanother-guy left review comments
@ntishkevichntishkevichntishkevich approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

needs proofreadA read-through is still needed to release the document

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@rlax@ntishkevich@another-guy@angryermine@lex111@gcor