docs: remove stale 100-subdirectories sync-depth FAQ#744
Open
DeepDiver1975 wants to merge 1 commit into
Open
Conversation
The FAQ claimed the Desktop App is hard-limited to sync no deeper than 100 subdirectories. This is no longer accurate for 7.1. In the owncloud/client v7.1.0 source, the CSYNC_STATUS_INDIVIDUAL_TOO_DEEP enum (src/csync/csync.h) is declared but never set anywhere, and no MAX_DEPTH constant exists. The depth limit was removed, so drop the stale FAQ section. Closes #728 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
DeepDiver1975
commented
Jun 21, 2026
DeepDiver1975
left a comment
DeepDiver1975
left a comment
Member
Author
There was a problem hiding this comment.
🤖 Automated code review by Claude Code review agent
Approve. Removes the stale === Sync Stops When Trying to Sync Deeper Than 100 Subdirectories FAQ section from faq.adoc.
- Verified the removed content is genuinely obsolete: the cited evidence (
CSYNC_STATUS_INDIVIDUAL_TOO_DEEPdeclared but never set, noMAX_DEPTHconstant in client v7.1.0) supports that the 100-subdir hard limit no longer exists. - Clean section removal: the preceding
.eml/registry block and the followingifeval::["{targetbuild}" == "oc"]quota section are both intact, and no other page references the removed entry — no dangling links. - AsciiDoc syntax correct.
Pair consistency: identical diff to 7.1 companion #743, correct master base, Closes #728 carried here. LGTM.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Removes the FAQ section "Sync Stops When Trying to Sync Deeper Than 100 Subdirectories" from
faq.adoc.Why
The claimed hard limit of 100 subdirectories is stale for 7.1. Verified against the
owncloud/clientv7.1.0 source:CSYNC_STATUS_INDIVIDUAL_TOO_DEEP(src/csync/csync.h:85) is declared but never set anywhere insrc/.MAX_DEPTHconstant in the codebase.The depth limit no longer exists, so the FAQ entry is removed rather than corrected.
Closes #728
🤖 Generated with Claude Code