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.

Sign up for GitHub

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

Jump to bottom

"waiting for all" definition changed #1363

Closed
domenic opened this issue Oct 26, 2018 · 0 comments · Fixed by #1463
Closed

"waiting for all" definition changed #1363

domenic opened this issue Oct 26, 2018 · 0 comments · Fixed by #1463

Comments

@domenic
Copy link
Contributor

domenic commented Oct 26, 2018

See w3ctag/promises-guide#55. I think the only change is you want to use ""get a promise for waiting for all" instead of "waiting for all". But now your spec won't have the problems mentioned in w3ctag/promises-guide#45!
@Ms2ger Ms2ger mentioned this issue Aug 19, 2019
Closed
Ms2ger added a commit to Ms2ger/ServiceWorker that referenced this issue Aug 21, 2019
@Ms2ger
Editorial: Update 'wait for all' usage.
01b0368 
Fixes w3c#1363, w3c#1461.
Ms2ger added a commit to Ms2ger/ServiceWorker that referenced this issue Aug 21, 2019
@Ms2ger
Editorial: Update 'wait for all' usage.
da54258 
Fixes w3c#1363.
Fixes w3c#1461.
@Ms2ger Ms2ger mentioned this issue Aug 21, 2019
Merged
mfalken pushed a commit that referenced this issue Sep 3, 2019
@Ms2ger @mfalken
Editorial: Update 'wait for all' usage. (#1463)
278c48b 
Fixes #1363.
Fixes #1461.

The authors of this commit are Ms2ger and Jake Archibald.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Editorial: Update 'wait for all' usage. Ms2ger/ServiceWorker
1 participant
@domenic