Closed
Bug 853222
Opened 11 years ago
Closed 11 years ago
Consider defereds that resolve async
Categories
(Add-on SDK Graveyard :: General, defect)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: irakli, Unassigned)
References
Details
This is mainly for discussing purposes rather than anything else. It's being brought up server times, let's agree on something and implement that so we can move on.
Reporter | ||
Comment 1•11 years ago
|
||
If users need that I'd suggest creating another module that uses promise/core under the hood and exposes same API with a difference that `resolve` will be delayed for one tick.
Comment 3•11 years ago
|
||
(In reply to Irakli Gozilalishvili [:irakli] [:gozala] [@gozala] from comment #1) > If users need that I'd suggest creating another module that uses > promise/core under the hood and exposes same API with a difference that > `resolve` will be delayed for one tick. Once of the common issues that people have with promise/core is hard-to-follow code, so having adding an extra wrapper might make things worse here. Given that testing and stability are easy (there are plenty of test suites to use) I don't have a problem with multiple implementations here so long as we can agree on a core API.
Needinfo-ing Mossop to make sure he gets to this.
Flags: needinfo?(dtownsend+bugmail)
Comment 5•11 years ago
|
||
We considered this in the email discussion and are working on making changes to the promises library in bug 810490
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(dtownsend+bugmail)
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•