853222 - Consider defereds that resolve async

Status: RESOLVED WORKSFORME

(Add-on SDK Graveyard :: General, defect)

Description

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

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.

Comment 1

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

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 2

[Richard Newman [:rnewman]]

This is relevant to FHR.

Comment 3

[Joe Walker [:jwalker]]

(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.

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Needinfo-ing Mossop to make sure he gets to this.

Comment 5

[Dave Townsend [:mossop]]

We considered this in the email discussion and are working on making changes to the promises library in bug 810490