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.

Sign up for GitHub

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

Jump to bottom

[New Arch] Remove file/folder operation #2864

Closed
16 of 20 tasks
abelgardep opened this issue Apr 24, 2020 · 0 comments · Fixed by #3214
Closed
16 of 20 tasks

[New Arch] Remove file/folder operation #2864

abelgardep opened this issue Apr 24, 2020 · 0 comments · Fixed by #3214
Assignees
@abelgardep
Labels
Estimation - 3 (M) New architecture
Milestone
3.0 - Synchroniza...

Comments

@abelgardep
Copy link
Contributor

abelgardep commented Apr 24, 2020 •
edited
Loading

Synchronization epic: #2818

AC:

  • Support removing the content in local (if downloaded), remote or both

TASKS

  • Research (if needed)
  • Create branch new_sync/remove_file
  • Development tasks
    • Create new usecase -> RemoveFileUseCase
    • Add removeFile to repository
    • Implement removeFile
    • Add removeFile to RemoteDataSource (Call CheckRemotePath before RemoveRemoteFileOperation to check if folder already exists?)
    • Migrate RemoveRemoteFileOperation to kotlin
    • Migrate RemoveRemoteChunksFolderOperation to kotlin
    • Add removeFile to current FileService
    • Create a new ChunkService to handle everything related to chunks
    • Remove local file
    • Remove file from database
    • Update DocumentsProvider
    • Error handling
    • Unit tests
  • Code review and apply changes requested
  • Design test plan
  • QA
  • Merge branch new_sync/remove_file into new_arch/sync

PR
@davigonz davigonz mentioned this issue Apr 24, 2020
Closed
75 tasks
@jesmrec jesmrec added this to the Synchronization milestone milestone Jun 19, 2020
@abelgardep abelgardep mentioned this issue Sep 10, 2020
Closed
11 tasks
@abelgardep abelgardep self-assigned this Apr 16, 2021
This was referenced Apr 29, 2021
Merged
Merged
@abelgardep abelgardep linked a pull request Apr 29, 2021 that will close this issue
[New arch] Remove file #3214
Merged
@jesmrec jesmrec removed the Sprint label May 7, 2021
This was referenced May 11, 2021
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abelgardep abelgardep

Labels
Estimation - 3 (M) New architecture
Projects
None yet
Milestone
3.0 - Synchronization milestone
Development

Successfully merging a pull request may close this issue.

[New arch] Remove file owncloud/android
2 participants
@jesmrec @abelgardep