Skip to content

feat(sync): download folder including all nested files#16744

Open
alperozturk96 wants to merge 7 commits intomasterfrom
feat/sync-all
Open

feat(sync): download folder including all nested files#16744
alperozturk96 wants to merge 7 commits intomasterfrom
feat/sync-all

Conversation

@alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Mar 24, 2026

Issue: #269

Any feedback regarding icons and wording?

@AndyScherzinger @kra-mo

Image 2 Image 1 Image 3

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@alperozturk96 alperozturk96 linked an issue Mar 24, 2026 that may be closed by this pull request
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@alperozturk96 alperozturk96 added this to the Nextcloud App 33.1.0 milestone Mar 24, 2026
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@github-actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/16744.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@github-actions
Copy link

blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed.

@github-actions
Copy link

Codacy

SpotBugs

CategoryBaseNew
Bad practice3939
Correctness7474
Dodgy code236235
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3434
Performance4343
Security1818
Total454453

@AndyScherzinger
Copy link
Member

Icon looks okay to me, didn't find any thats would be a better option match, maybe @kra-mo has an idea.

Other than that for me the question would be the position in the list. I would put it below the sync item, so it is close in terms of feature. It does however raise the question why there is is sync and sync all.

@alperozturk96
Copy link
Collaborator Author

Icon looks okay to me, didn't find any thats would be a better option match, maybe @kra-mo has an idea.

Other than that for me the question would be the position in the list. I would put it below the sync item, so it is close in terms of feature. It does however raise the question why there is is sync and sync all.

Sync -> Downloads first hierarchy
Sync all -> Downloads entire folder hiearchy

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sync folders + subfolders

2 participants