Skip to content

fix(discovery): do not sync shares with "Allow download and sync" unset - #9963

Merged
nilsding merged 4 commits into
masterfrom
bugfix/noid/no-download-and-sync-folders
May 19, 2026
Merged

fix(discovery): do not sync shares with "Allow download and sync" unset#9963
nilsding merged 4 commits into
masterfrom
bugfix/noid/no-download-and-sync-folders

Conversation

@nilsding

@nilsding nilsding commented May 4, 2026

Copy link
Copy Markdown
Member

The option does not only apply to files, but also to folders.

Also: do not display shared items without that option in Windows on-demand folders

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@nilsding

nilsding commented May 4, 2026

Copy link
Copy Markdown
Member Author

/backport to stable-33.0

@nilsding
nilsding force-pushed the bugfix/noid/no-download-and-sync-folders branch from cde4c08 to a7f0efb Compare May 13, 2026 12:43
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)
98 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@nilsding
nilsding force-pushed the bugfix/noid/no-download-and-sync-folders branch 2 times, most recently from cc096f3 to bc3c581 Compare May 18, 2026 09:46
nilsding added 4 commits May 19, 2026 09:36
… permission

The permission does not only apply to files, but also to folders.

Signed-off-by: Jyrki Gadinger <[email protected]>
otherwise cfapi would list folders that are skipped by the sync
engine itself

Signed-off-by: Jyrki Gadinger <[email protected]>
@nilsding
nilsding force-pushed the bugfix/noid/no-download-and-sync-folders branch from bc3c581 to 774e04e Compare May 19, 2026 07:36
@github-actions

Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-9963.zip

Digest: sha256:9c58c6a02d63f02ecb24cef18717dae34cc14df8c7facee206ba30ec88b0557f

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@nilsding
nilsding merged commit 2153634 into master May 19, 2026
20 of 21 checks passed
@nilsding
nilsding deleted the bugfix/noid/no-download-and-sync-folders branch May 19, 2026 08:13
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.

2 participants