Opened 13 years ago

Closed 6 years ago

#1149 closed enhancement (wontfix)

notify when queued request is fired

Reported by: Grzegorz Tańczky Owned by: Bryan Forbes
Priority: high Milestone: future
Component: IO Version: 0.3.1
Keywords: iframeio bind io request queue queued needsreview Cc:
Blocked By: Blocking:

Description (last modified by James Burke)

I suggest to add callback function whch is fired after this line:

if((this.currentRequest)
(this.requestQueue.length == 0)){ return; }

in fireNextRequest function

it is useful to be informed when request is fired, for example in fileuploads using IFrameIO

Change History (12)

comment:1 Changed 13 years ago by dylan

Milestone: 0.4

comment:2 Changed 13 years ago by James Burke

I might be able to pick this one up when I do the work to add timeout support for IframeIO. Perhaps add an additional callback handler that is supported on the dojo.io.Request object -- start: function(type, kwArgs). Not sure start is the best word for the function. Trying to keep it short like "load" or "handle". started? begin? Any better ideas?

Also, perhaps instead of using its own private queue, should IframeIO use dojo.io.queueBind() in some way? I haven't looked at the code yet, just making some notes, asking for feedback.

comment:3 Changed 13 years ago by dylan

Owner: changed from alex to James Burke

comment:4 Changed 13 years ago by dylan

James, feature freeze for 0.4 is end of Monday... let me know if you want to tackle this before then... otherwise, please push to 0.5.

comment:5 Changed 13 years ago by dylan

Milestone: 0.40.5

comment:6 Changed 12 years ago by James Burke

Milestone: 0.91.0

Not critical for 0.9. Moving to 1.0

comment:7 Changed 12 years ago by James Burke

Milestone: 1.02.0
Version: browserio_package0.3.1

comment:8 Changed 12 years ago by alex

Milestone: 2.01.3

Milestone 2.0 deleted

comment:9 Changed 11 years ago by James Burke

Description: modified (diff)
Milestone: 1.3future

comment:10 Changed 8 years ago by ben hockey

Keywords: needsreview added

should we close this ticket?

comment:11 Changed 8 years ago by bill

Owner: changed from James Burke to Bryan Forbes

Bulk change to reassign IO tickets to Bryan, since he is working on new dojo/request module. Some of these tickets should probably be closed as already fixed, invalid, or wontfix.

comment:12 Changed 6 years ago by dylan

Resolution: wontfix
Status: newclosed

This isn't really possible/relevant with how IO works now (deferreds fire once). Hopefully the flexibility of dojo/request is sufficient.

Closing this out as wontfix.

Note: See TracTickets for help on using tickets.