Skip to content
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

Cancel uploads #175

Open
2 tasks done
jesmrec opened this issue Nov 29, 2018 · 5 comments
Open
2 tasks done

Cancel uploads #175

jesmrec opened this issue Nov 29, 2018 · 5 comments
Labels
feature:fileList feature:uploads all kind of uploads p3-medium Normal priority sdk-involved Requires SDK development or discussion

Comments

@jesmrec
Copy link
Contributor

jesmrec commented Nov 29, 2018

As user, i want to cancel a progressing upload, so that i can save time and bandwidth if i select incorrectly a file to upload

Option to cancel a pending/ongoing upload is a good choice if the user select a wrong file or simply decides to put off the upload. Actually, uploads can not be cancelled and user must wait till upload finishes to get rid of the file. In case the file is huge, time and bandwidth are wasted.

Taking advantage of the lack of three-dot-button in cells of uploading files, set here a cancel option in the same place is a good choice.

AC:

  • In the list of files, the uploading file should display an option to cancel (activity involved?)
  • If the user cancels, upload stops and file is removed from the bookmark.
  • If the user cancels offline, upload process is dismissed and not resumed when user recovers connectivity

Operations done:

  • Uploads
  • Downloads
@jesmrec jesmrec added this to the 0.3.0 milestone Nov 29, 2018
@jesmrec jesmrec added the sdk-involved Requires SDK development or discussion label Dec 4, 2018
@felix-schwarz
Copy link
Contributor

Yes, the SDK supports cancelling uploads by calling cancel on the NSProgress objects returned by the core. That cancel is passed along to the actual HTTP request in the background NSURLSession and cancels the upload. I'll have to check how the Sync Engine deals with that event, though.

@michaelstingl michaelstingl modified the milestones: 0.3.0, 0.4.0 Jan 4, 2019
@michaelstingl michaelstingl added the p3-medium Normal priority label Jan 22, 2019
@felix-schwarz
Copy link
Contributor

As of 1a7f260 there's now some initial support for cancelling uploads inline as well as in the activity tab:
bildschirmfoto 2019-01-29 um 00 34 08

@jesmrec
Copy link
Contributor Author

jesmrec commented Feb 22, 2019

After implementation on #253, it is already implemented for uploads/downloads and pending for a possible future implementation on other operations. Removing from current milestone, waiting for be triaged again

@jesmrec jesmrec removed this from the 1.0.0 milestone Feb 22, 2019
@michaelstingl
Copy link
Contributor

@jesmrec what are the remaining tasks?

@jesmrec
Copy link
Contributor Author

jesmrec commented Feb 22, 2019

with the latest implementation:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature:fileList feature:uploads all kind of uploads p3-medium Normal priority sdk-involved Requires SDK development or discussion
Projects
None yet
Development

No branches or pull requests

3 participants