Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit 5dea441

Browse files
committed
minor fixes
1 parent 0760c90 commit 5dea441

File tree

1 file changed

+5
-2
lines changed
  • 1-js/11-async/08-async-await/04-promise-all-failure

1 file changed

+5
-2
lines changed

‎1-js/11-async/08-async-await/04-promise-all-failure/solution.md‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ It may be important that all queries complete, especially if some of them make i
4646

4747
So we should wait until all promises are settled before going further with the execution and eventually disconnecting.
4848

49-
Here's another implementation. It also resolves with the first error, but waits until all promises are settled.
49+
Here's another implementation. It behaves similar to `Promise.all` - also resolves with the first error, but waits until all promises are settled.
5050

5151
```js
5252
function customPromiseAllWait(promises) {
@@ -82,4 +82,7 @@ function customPromiseAllWait(promises) {
8282

8383
Now `await customPromiseAllWait(...)` will stall the execution until all queries are processed.
8484

85-
This is the most reliable approach.
85+
This is a more reliable approach.
86+
87+
Lastly, if we'd like to know about all the errors, e.g. for logging purposes, we can use `Promise.allSettled`.
88+

0 commit comments

Comments
(0)

AltStyle によって変換されたページ (->オリジナル) /