Skip to content
This repository has been archived by the owner on Jul 30, 2018. It is now read-only.

Add upload progress reporting #285

Closed
rorticus opened this issue Feb 10, 2017 · 1 comment · Fixed by #326
Closed

Add upload progress reporting #285

rorticus opened this issue Feb 10, 2017 · 1 comment · Fixed by #326
Assignees
@rorticus
Milestone
2017.05

Comments

@rorticus
Copy link
Contributor

Once #261 lands, we should add upload progress reporting to the xhr and node providers. Currently, they only provide reporting on download progress.
@rorticus rorticus mentioned this issue Feb 10, 2017
Merged
@dylans dylans added this to the 2017.03 milestone Mar 6, 2017
@dylans dylans modified the milestones: 2017.04, 2017.03 Mar 25, 2017
@dylans dylans added the beta2 label Mar 25, 2017
@rorticus rorticus self-assigned this Apr 7, 2017
rorticus added a commit to rorticus/core that referenced this issue Apr 21, 2017
@rorticus
Adding XHR upload progress reporting, issue dojo#285
32accf9
rorticus added a commit to rorticus/core that referenced this issue Apr 28, 2017
@rorticus
Adding upload progress reporting, issue dojo#285
3a17583
rorticus added a commit to rorticus/core that referenced this issue Apr 28, 2017
@rorticus
Adding upload progress reporting, issue dojo#285
a10dc18
rorticus added a commit to rorticus/core that referenced this issue Apr 28, 2017
@rorticus
Adding upload progress reporting, issue dojo#285
0b7fa90
@rorticus rorticus mentioned this issue Apr 28, 2017
Merged
3 tasks
@kitsonk
Copy link
Member

kitsonk commented Apr 28, 2017 •
edited
Loading

Sorry for potentially changing tact on this, but is modelling on events the right paradigm? This seems like a perfect use case for Observable. Even the name of the proposed PR seems to indicate that is the right paradigm. 😁

@dylans dylans modified the milestones: 2017.04, 2017.05 Apr 29, 2017
rorticus added a commit to rorticus/core that referenced this issue May 5, 2017
@rorticus
Using observables for all request/response events, issue dojo#285
edd9752
rorticus added a commit to rorticus/core that referenced this issue May 5, 2017
@rorticus
Using observables for all request/response events, issue dojo#285
b6749ed
rorticus added a commit to rorticus/core that referenced this issue May 12, 2017
@rorticus
Adding upload progress reporting, issue dojo#285
a9c9ae4
rorticus added a commit to rorticus/core that referenced this issue May 12, 2017
@rorticus
Using observables for all request/response events, issue dojo#285
bce9996
rorticus added a commit that referenced this issue May 12, 2017
@rorticus
Adding upload progress reporting, issue #285 (#326)
a51adb2 
* Adding upload progress reporting, issue #285

* Using observables for all request/response events, issue #285

* Making SubscriptionPool queueing

* More test coverage

* More tests

* Better coverage for QueuingEvented

* Updating test coverage
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rorticus rorticus

Labels
None yet
Projects
None yet
Milestone
2017.05
Development

Successfully merging a pull request may close this issue.

Adding upload progress reporting, issue #285 rorticus/core
3 participants
@dylans @kitsonk @rorticus