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

[FEATURE REQUEST] Improve the removed from original folder option in Auto Uploads #4357

Closed
11 tasks done
jesmrec opened this issue Apr 4, 2024 · 1 comment · Fixed by #4437
Closed
11 tasks done

Comments

@jesmrec
Copy link
Collaborator

jesmrec commented Apr 4, 2024

The option removed from original folder in auto uploads is potentially destructive

Screenshot 2024-04-04 at 13 40 53

Removing the local file when worker moves the taken pictures to the scoped storage forces the uploads to work in order not to lose the pictures/videos. If upload fails for X reason and is not recoverable, the file is lost: it is neither in server, nor in local

Some solutions:

  • Adding a warning when option is clicked: The pictures/videos taken by the camera will not be available in your device anymore. This could be potentially dangerous in case of failure in the uploads because external reasons. In that way, problem is not solved but user is warned about the existing perils.

  • Removing the local file only when the upload is finished. Not sure how to control this, unless different kinds of uploads are distinguishable.

  • Other ideas...

TASKS

  • Research (if needed)
  • Create branch feature/feature_name
  • Development tasks
    • Create Work to remove local file
    • Enqueued two works: uploadFileFromContentUriWorker and removeLocalFileWorker.
    • Removed code relate to remove local file in UploadFileFromContentUriWorker
    • Implement unit tests (if needed)
  • Code review and apply changes requested
  • Design test plan
  • QA
  • Merge branch feature/feature_name into master
@JuancaG05
Copy link
Collaborator

Could be that auto-uploads are removed instantly because permissions are removed by Android over the files after a while... Needs to be checked

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants