-
Notifications
You must be signed in to change notification settings - Fork 181
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.
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
Adding support for whenSettled to DeferredManager (Promise.allSettled) #118
Conversation
This is nice! We'll likely move this to 2.x. It's on the roadmap here: That being said, some of the breaking changes were significant. Would it be possible to only have whenSettled(...) method for the 2.x branch? |
Indeed, the changes make more sense for the |
5e2390c
to
6076c0a
Compare
@saturnism There shouldn't be any breaking changes. Which changes are you referring to? |
I believe @saturnism is referring to the breaking changes we made in the |
Hey,
in addition to my codestyle PR I have added support for whenSettled to DeferredManager and its children. This is basically equivalent to Promise.allSettled. whenSettled waits for all Promises to either get resolved or rejected and then returns all results in the correct order.
Current functionality should remain untouched by my commits.
I have compiled and tested jdeferred in my Android App project.
This change is