Skip to content

File and FileReader#215

Merged
javascript-translate-bot merged 10 commits into
javascript-tutorial:masterfrom
shchegol:master
May 30, 2019
Merged

File and FileReader#215
javascript-translate-bot merged 10 commits into
javascript-tutorial:masterfrom
shchegol:master

Conversation

@shchegol

Copy link
Copy Markdown
Contributor

Предлагаю следующий вариант перевода.

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label May 29, 2019
@javascript-translate-bot javascript-translate-bot requested a review from a team May 29, 2019 12:29
@javascript-translate-bot

Copy link
Copy Markdown
Contributor

Error: the article has another translator @shchegol in the Progress Issue #60. Please update the progress issue or ask @translate-ru to do so.

@shchegol

shchegol commented May 29, 2019

Copy link
Copy Markdown
Contributor Author

@iliakan

  1. В статье ничего нет о том, что FileReader читает содержимое файлов асинхронно.
  2. Для FileReader вместо w3 можно дать ссылку на русский MDN

Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels May 29, 2019
@javascript-translate-bot

Copy link
Copy Markdown
Contributor

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@shchegol

Copy link
Copy Markdown
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels May 29, 2019
Comment thread 4-binary/04-file/article.md Outdated
@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels May 29, 2019
@javascript-translate-bot

Copy link
Copy Markdown
Contributor

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@shchegol

Copy link
Copy Markdown
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels May 29, 2019
@javascript-translate-bot javascript-translate-bot added needs +1 One more review needed and removed review needed Review needed, please approve or request changes labels May 29, 2019
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
@javascript-translate-bot javascript-translate-bot added the changes requested Waiting for changes and /done from PR author label May 29, 2019
@javascript-translate-bot

Copy link
Copy Markdown
Contributor

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@shchegol

Copy link
Copy Markdown
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot requested a review from a team May 29, 2019 15:16
@shchegol

Copy link
Copy Markdown
Contributor Author

Чёт я не то сделал похоже...

@iliakan iliakan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Спасибо, пожалуйста, внеси небольшие изменения =)

Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
Comment thread 4-binary/04-file/article.md Outdated
@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels May 29, 2019
@javascript-translate-bot

Copy link
Copy Markdown
Contributor

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@shchegol

Copy link
Copy Markdown
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels May 30, 2019
@javascript-translate-bot javascript-translate-bot requested a review from a team May 30, 2019 06:39
@iliakan

iliakan commented May 30, 2019

Copy link
Copy Markdown
Member

Разрешил конфликты.
Не понимаю, почему done, ведь правки не были учтены, посмотри их плиз @shchegol

@iliakan iliakan added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels May 30, 2019
@iliakan

iliakan commented May 30, 2019

Copy link
Copy Markdown
Member

@shchegol в статье есть про то, что данные приходят через события (=> асинхронно), этого не хватает?

@shchegol

shchegol commented May 30, 2019

Copy link
Copy Markdown
Contributor Author

@iliakan

  1. Я сейчас всё перепроверил, не могу найти, какие исправления я ещё не учёл? В последнем 726d889 коммите я вроде бы все предложенные тобой исправления внёс.
  2. Про эту фразу: Данные передаются при помощи событий, так как чтение с диска может занять время. - вот сейчас я до конца понял что имелось ввиду, сначала нет.

Comment thread 4-binary/04-file/article.md Outdated
@javascript-translate-bot

Copy link
Copy Markdown
Contributor

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@shchegol

Copy link
Copy Markdown
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels May 30, 2019
@javascript-translate-bot javascript-translate-bot requested a review from a team May 30, 2019 09:09
@iliakan

iliakan commented May 30, 2019

Copy link
Copy Markdown
Member

🚀

@javascript-translate-bot javascript-translate-bot merged commit 2651b4a into javascript-tutorial:master May 30, 2019
@javascript-translate-bot

Copy link
Copy Markdown
Contributor

Error: the article has another translator @shchegol , not @shchegol in the Progress Issue #60. Please update the progress issue or ask maintainers to do so.

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

Labels

needs +1 One more review needed review needed Review needed, please approve or request changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants