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

[New Arch] Copy file/folder operation #2866

Closed
12 of 16 tasks
abelgardep opened this issue Apr 24, 2020 · 2 comments · Fixed by #3253
Closed
12 of 16 tasks

[New Arch] Copy file/folder operation #2866

abelgardep opened this issue Apr 24, 2020 · 2 comments · Fixed by #3253

Comments

@abelgardep
Copy link
Contributor

abelgardep commented Apr 24, 2020

Synchronization epic: #2818

AC:

  • Copy remote in server.
  • Local: check if file is downloaded to move to new location. Check if new version is available in case of file already downloaded.
  • In case of copy folder: check if items inside are downloaded, if they have newer versions, update.
  • Check if the copied file is downloaded, in case the source one is downloaded.

QA Checks

https://github.com/owncloud/QA/blob/master/Mobile/Android/Sync/Copy.md

TASKS

  • Research (if needed)
  • Create branch new_sync/copy
  • Development tasks
    • Create new usecase -> CopyFileUseCase
    • Add copyFile to repository
    • Add copyFile to RemoteDataSource (Call CheckRemotePath before CopyRemoteFileOperation to check if folder already exists?)
    • Migrate CopyRemoteFileOperation to kotlin
    • Update database
    • Copy local files/folders
    • Update DocumentsProvider
    • Error handling
    • Unit tests
  • Code review and apply changes requested
  • Design test plan
  • QA
  • Merge branch new_sync/copy into new_arch/sync

PR

@abelgardep abelgardep changed the title [FEATURE REQUEST] Copy file/folder operation [New Arch] Copy file/folder operation Apr 24, 2020
@jesmrec jesmrec added this to the Synchronization milestone milestone Jun 19, 2020
This was referenced May 21, 2021
@abelgardep abelgardep self-assigned this May 21, 2021
@abelgardep abelgardep linked a pull request May 21, 2021 that will close this issue
@abelgardep
Copy link
Contributor Author

Related to AC:

Copy remote in server.
Local: check if file is downloaded to move to new location. Check if new version is available in case of file already downloaded.
In case of copy folder: check if items inside are downloaded, if they have newer versions, update.
Check if the copied file is downloaded, in case the source one is downloaded.

I think that it could be enough to copy files remotely and to update the database accordingly. We could avoid some problems that way instead of copying every downloaded file.

@abelgardep
Copy link
Contributor Author

Will be tested within synchronization epic

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.

2 participants