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.

Sign up for GitHub

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

Jump to bottom

Translate "AJAX and APIs" into Russian #30

Merged
merged 3 commits into from
Feb 10, 2019
Merged

Translate "AJAX and APIs" into Russian #30

merged 3 commits into from
Feb 10, 2019

Conversation

avevlad
Copy link
Contributor

@avevlad avevlad commented Feb 9, 2019

No description provided.

@avevlad avevlad mentioned this pull request Feb 9, 2019
Closed
@lex111 lex111 requested a review from gaearon February 9, 2019 19:33
gaearon
gaearon requested changes Feb 9, 2019
View reviewed changes
Copy link
Member

@gaearon gaearon 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/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
lex111
lex111 reviewed Feb 9, 2019
View reviewed changes
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
lex111
lex111 reviewed Feb 9, 2019
View reviewed changes
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
@avevlad
Copy link
Contributor Author

avevlad commented Feb 9, 2019

@gaearon @lex111 @ModPhoenix правки внесены!

@tesseralis tesseralis mentioned this pull request Feb 9, 2019
Closed
88 tasks
@lex111 lex111 added the needs review A pull request ready to be reviewed label Feb 9, 2019
gaearon
gaearon approved these changes Feb 10, 2019
View reviewed changes
Copy link
Member

@gaearon gaearon 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/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
content/docs/faq-ajax.md Outdated Show resolved Hide resolved
@gaearon gaearon added work in progress Author opened the pull-request but it's still being worked on and removed needs review A pull request ready to be reviewed labels Feb 10, 2019
@avevlad
Copy link
Contributor Author

avevlad commented Feb 10, 2019

@gaearon так и правда лучше, внес правки.

@gaearon gaearon merged commit 007d2a3 into reactjs:master Feb 10, 2019
@gaearon
Copy link
Member

gaearon commented Feb 10, 2019

Спасибо!

@gaearon gaearon removed the work in progress Author opened the pull-request but it's still being worked on label Feb 10, 2019
@avevlad
Copy link
Contributor Author

avevlad commented Feb 10, 2019

@gaearon @lex111 @ModPhoenix Спасибо!

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

@ModPhoenix ModPhoenix ModPhoenix left review comments

@lex111 lex111 lex111 approved these changes

@gaearon gaearon gaearon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@avevlad @gaearon @lex111 @ModPhoenix