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 AJAX and APIs #25

Closed
wants to merge 1 commit into from
Closed

Translate AJAX and APIs #25

wants to merge 1 commit into from

Conversation

lex111
Copy link
Member

@lex111 lex111 commented Feb 9, 2019

No description provided.

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.

Accepted with nits


The component below demonstrates how to make an AJAX call in `componentDidMount` to populate local component state.
Компонент ниже показывает, как сделать AJAX-запрос в методе `componentDidMount`, чтобы наполнить локальное состояние компонента.
Copy link
Member

Choose a reason for hiding this comment

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

#27


The example API returns a JSON object like this:
Пример нашего API возвращает следующий JSON-объект:
Copy link
Member

Choose a reason for hiding this comment

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

"Пример API" звучит странно. Может

Допустим, наш API возвращает следующий JSON-объект:

(Оригинал тоже немножко странный но по-английски это понятнее было)

// instead of a catch() block so that we don't swallow
// exceptions from actual bugs in components.
// Примечание: важно обрабатывать ошибки в этом колбэке,
// а не использовать блок catch() для этого, чтобы не
Copy link
Member

Choose a reason for hiding this comment

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

"для этого" можно убрать

// Примечание: важно обрабатывать ошибки в этом колбэке,
// а не использовать блок catch() для этого, чтобы не
// перехватывать исключения из-за существующих багов
// в компонентах.
Copy link
Member

Choose a reason for hiding this comment

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

"существующих багов" -> может "настоящих багов"? Вопрос не столько про то, были они уже или не были.

@lex111
Copy link
Member Author

lex111 commented Feb 9, 2019

Я ступил и перевёл страницу, которая была уже забита :(

Поэтому этот PR будет закрыт, после создания перевода от @avevlad.

@avevlad
Copy link
Contributor

avevlad commented Feb 9, 2019

@lex111 почему не закрыть его сейчас?

@lex111
Copy link
Member Author

lex111 commented Feb 9, 2019

@avevlad чтобы я потом сослался на твой.

@lex111 lex111 added the duplicate This issue or pull request already exists label Feb 9, 2019
@lex111
Copy link
Member Author

lex111 commented Feb 9, 2019

Разумеется, можешь свободно использовать этот перевод, учитывая предложенные улучшения.

@avevlad
Copy link
Contributor

avevlad commented Feb 9, 2019

@lex111 создал #30

@lex111
Copy link
Member Author

lex111 commented Feb 9, 2019

Закрыто в пользу #30

@lex111 lex111 closed this Feb 9, 2019
@lex111 lex111 deleted the faq-ajax branch February 9, 2019 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants