Skip to content

Fetch: Download progress - #434

Open
mosioc wants to merge 1 commit into
javascript-tutorial:masterfrom
mosioc:docs-translate-fetch-download-progress
Open

Fetch: Download progress#434
mosioc wants to merge 1 commit into
javascript-tutorial:masterfrom
mosioc:docs-translate-fetch-download-progress

Conversation

@mosioc

@mosioc mosioc commented Jun 18, 2026

Copy link
Copy Markdown

No description provided.

@javascript-translate-bot

Copy link
Copy Markdown

Error: the article already has PR number in the Progress Issue #1, it's 434 ⁉️

1 similar comment
@javascript-translate-bot

Copy link
Copy Markdown

Error: the article already has PR number in the Progress Issue #1, it's 434 ⁉️

@javascript-translate-bot

Copy link
Copy Markdown

Error: the article already has PR number in the Progress Issue #1, it's 434 ⁉️

1 similar comment
@javascript-translate-bot

Copy link
Copy Markdown

Error: the article already has PR number in the Progress Issue #1, it's 434 ⁉️

@CLAassistant

CLAassistant commented Jun 18, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@mosioc

mosioc commented Jun 20, 2026

Copy link
Copy Markdown
Author

@mahdyar

@mosioc

mosioc commented Aug 4, 2026

Copy link
Copy Markdown
Author

Hey. anyone?
@mahdyar

@mahdyar mahdyar 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.

Hi @mosioc, and sorry for the delayed review.
Unfortunately line numbers don't match and it'll cause more problems in future when syncing with upstream, so please fix it and ask for another review.
Thank you for your contribution!

- **`done`** -- `true` when the reading is complete, otherwise `false`.
- **`value`** -- a typed array of bytes: `Uint8Array`.
نتیجه‌ی فراخوانی `await reader.read()` یک شیء با دو ویژگی است:

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.

You added an extra line here for example.

// مرحله 3: خواندن داده
let receivedLength = 0; // تا این لحظه این مقدار بایت دریافت شده
let chunks = []; // آرایه‌ای از تکه‌های باینری دریافت‌شده (بدنه)

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.

same here

@javascript-translate-bot

Copy link
Copy Markdown

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

4 similar comments
@javascript-translate-bot

Copy link
Copy Markdown

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

@javascript-translate-bot

Copy link
Copy Markdown

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

@javascript-translate-bot

Copy link
Copy Markdown

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

@javascript-translate-bot

Copy link
Copy Markdown

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants