docs: remove stale 100-subdirectories sync-depth FAQ#744
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
left a comment
DeepDiver1975
left a comment
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.
dj4oC
left a comment
dj4oC
left a comment
There was a problem hiding this comment.
Removes the stale "Sync Stops When Trying to Sync Deeper Than 100 Subdirectories" FAQ entry from faq.adoc, since the underlying hard limit no longer exists in the client.
Findings
Nothing blocking. Removal is a clean section deletion; the surrounding ifeval::["{targetbuild}" == "oc"] block remains intact and no other page links to the removed section.
Verdict
Approved
🤖 Automated review by Claude Code
Generated by Claude Code
2 participants
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