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] Upload a single file using new architecture #3424

Closed
11 of 20 tasks
jesmrec opened this issue Oct 27, 2021 · 2 comments · Fixed by #3686
Closed
11 of 20 tasks

[FEATURE REQUEST] Upload a single file using new architecture #3424

jesmrec opened this issue Oct 27, 2021 · 2 comments · Fixed by #3686

Comments

@jesmrec
Copy link
Collaborator

jesmrec commented Oct 27, 2021

This flow to accomplish:

  1. User opens FAB and select upload file
  2. Show the user the document provider (SAF), to select a file from there. Only one file for the moment. multiselection, a N2H.
  3. Fetch the selected file
  4. Send to the server, inside the correct location updating DB
  5. New file is visible in the server (check in web or other client)

errors, notifications and upload view to be done in the following issues.

TASKS

  • Research (if needed)
  • Create branch new_arch/upload_single_file
  • Development tasks
    • New usecase to upload files from everywhere.
    • Implement force overwrite then uploading files
    • Define exactly what is the LOCAL_BEHAVIOR, document, and rename them to a more understandable KEY
    • Remove legacy create remote folder key, since it is not used anymore.
    • Generate a new ocfile to upload
    • Check if chunking is allowed or not and prepare the code to trigger a chunked upload
    • Generate OCUpload and store it in the database
    • Add a listener and update the upload progress similar to downloads
    • Check if we can automatize upload retries when connection fails
    • Use new usecase to upload files from camera uploads
    • Use new usecase to upload files from manual uploads
    • Use new usecase to upload files after receiving external files
    • Use new usecase to upload files from documentsprovider
  • Code review and apply changes requested
  • Design test plan
  • QA
  • Merge branch new_arch/upload_single_file into master

PR

@abelgardep
Copy link
Contributor

Known issues

@abelgardep
Copy link
Contributor

We can close this at the moment. Already merged into the synchronization branch. We will keep improving the uploads behavior in upcoming PRs

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