docs: remove stale 100-subdirectories sync-depth FAQ (7.1)#743
Open
DeepDiver1975 wants to merge 1 commit into
Open
docs: remove stale 100-subdirectories sync-depth FAQ (7.1)#743DeepDiver1975 wants to merge 1 commit into
DeepDiver1975 wants to merge 1 commit into
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. 7.1 port removing the stale 100-subdirectories sync-depth FAQ section from faq.adoc.
- Removed content verified obsolete (no depth limit /
MAX_DEPTHin client v7.1.0;CSYNC_STATUS_INDIVIDUAL_TOO_DEEPdeclared but unused). Surrounding sections remain intact, no dangling references. - AsciiDoc syntax correct.
Pair consistency: byte-identical change to master PR #744, correctly targets the 7.1 branch, uses Refs #728 (closing keyword on master PR). 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.
Refs #728 (master PR carries the closing keyword).
🤖 Generated with Claude Code