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] Move file/folder Operation #2865

Closed
17 of 19 tasks
abelgardep opened this issue Apr 24, 2020 · 2 comments · Fixed by #3232
Closed
17 of 19 tasks

[New Arch] Move file/folder Operation #2865

abelgardep opened this issue Apr 24, 2020 · 2 comments · Fixed by #3232
Assignees
@abelgardep
Labels
Estimation - 5 (L) New architecture
Milestone
3.0 - Synchroniza...

Comments

@abelgardep
Copy link
Contributor

abelgardep commented Apr 24, 2020 •
edited
Loading

Synchronization epic: #2818

TASKS

  • Research (if needed)
  • Create branch new_sync/move
  • Development tasks
    • Create new usecase -> MoveFileUseCase
    • Add moveFile to repository
    • Add moveFile to RemoteDataSource (Call CheckRemotePath before MoveRemoteFileOperation to check if folder already exists?)
    • Migrate MoveRemoteFileOperation to kotlin
    • Migrate MoveRemoteChunkFileOperation to kotlin
    • Add move to FileService
    • Add move to ChunkService
    • Update database
    • Move local files/folders
    • Update DocumentsProvider
    • Error handling
    • Unit tests
  • Code review and apply changes requested
  • Design test plan
  • QA
  • Merge branch new_sync/move 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 May 7, 2021
This was referenced May 11, 2021
Closed
Merged
Merged
@abelgardep abelgardep linked a pull request May 11, 2021 that will close this issue
[New arch] Move #3232
Merged
@abelgardep
Copy link
Contributor Author

Pending:

  • Check behaviour when moving Av. Offline or moving to a folder marked as Av. Offline
  • Check performance when moving A LOT OF files

@abelgardep abelgardep mentioned this issue May 19, 2021
Merged
@abelgardep
Copy link
Contributor Author

Will be tested within the synchronization epic.

@jesmrec jesmrec removed the Sprint label May 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abelgardep abelgardep

Labels
Estimation - 5 (L) New architecture
Projects
None yet
Milestone
3.0 - Synchronization milestone
Development

Successfully merging a pull request may close this issue.

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