FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

docs(fetch): add modern `Promise.allSettled` solution to fetch-users task by illia-m-b · Pull Request #3987 · javascript-tutorial/en.javascript.info · GitHub

docs(fetch): add modern Promise.allSettled solution to fetch-users task - #3987

Open
illia-m-b wants to merge 1 commit into
javascript-tutorial:masterfrom
illia-m-b:patch-1
Open

docs(fetch): add modern Promise.allSettled solution to fetch-users task#3987
illia-m-b wants to merge 1 commit into
javascript-tutorial:masterfrom
illia-m-b:patch-1

Conversation

Copy link
Copy Markdown

Closes #3986

Description

Appends a production-grade alternative solution to the "Fetch users from GitHub" task.

Changes Made

  • Appended an "Alternative modern approach" section to solution.md.
  • Added a code snippet utilizing Promise.allSettled, async callbacks, and object destructuring.

Technical Rationale

The original solution intentionally mixes .then() with await for educational purposes. This addition complements the lesson by demonstrating current industry best practices:

  • Pure async/await syntax without chained Promise methods.
  • Graceful fault tolerance for network failures via Promise.allSettled.
  • Independent, non-blocking fetch execution.

javascript-translate-bot added the review needed Review needed, please approve or request changes label Aug 5, 2026
javascript-translate-bot requested a review from a team August 5, 2026 18:13
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review needed Review needed, please approve or request changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proposal: Add modern Promise.allSettled alternative to "Fetch users" task solution

2 participants


Back | FazBrowse Home | New Git URL