Skip to content

feat(sync): show discovery phases and examined item counts - #11087

Open
norgan wants to merge 1 commit into
nextcloud:masterfrom
norgan:feat/discovery-progress-status
Open

norgan wants to merge 1 commit into
nextcloud:masterfrom
norgan:feat/discovery-progress-status

Conversation

@norgan

@norgan norgan commented Oct 16, 2026 •

Copy link
Copy Markdown

Fixing both lag and visibility of very large file stores

Implementation Detail
Preparation phases Starting, discovery and reconciliation
Examined counts Cumulative classified folders and files; separate from transfer totals
Update frequency Immediate first count; subsequent count updates at most once per 500 ms; immediate phase transitions
Progress bar Indeterminate during preparation; no percentage or ETA
Connection state Disconnected/retry labels
Regression corrections Known ignored-directory types retained; unknown skipped items excluded from counts; newly opened folder-model rows initialized
Tray updates Refresh the tray tooltip during starting, discovery and reconciliation; retain the existing transfer and completion behavior
Independent branch Base 5bf7bdf42c30d93f9896d8fcbe1c44bbc58640af; no batching change from #11086
Local Windows validation Result
Release client and test build Passed
DiscoveryProgressTest 12 passed, 0 failed, 0 skipped
RemoteDiscoveryTest 34 passed, 0 failed, 0 skipped
New regression cases against original submitted production code 4 failures reproduced
Tray tooltip regression against the prior published code 11 passed, 1 failed; corrected code: 12 passed, 0 failed
Tray tooltip integration test Actual Folder progress slot, dispatcher, ownCloudGui and QSystemTrayIcon tooltip exercised with an isolated fake account
Changed-line clang-tidy and clang-format Passed; no source changes
Isolated model/delegate previews Discovery, reconciliation, disconnected and retry states inspected
Combined guarded live build Counts displayed during discovery; Settings/Info navigation responsive
Live tooltip correction Isolated integration test passed; the existing live scan continues on the earlier protected package
Human review and DCO Reviewed and signed off
Complete large live sync Still running; completion unverified
Upstream-before UI screenshot Unavailable
macOS and Linux validation Not run in this Windows environment

AI assistance: Assisted-by: Codex:GPT-6 (implementation, tests, source analysis and validation).

@norgan
norgan force-pushed the feat/discovery-progress-status branch from e8710f9 to 494d826 Compare October 18, 2026 02:21
@norgan
norgan marked this pull request as ready for review October 18, 2026 02:21
@norgan
norgan marked this pull request as draft October 18, 2026 03:41
Assisted-by: Codex:GPT-6
Signed-off-by: Norgan <nath@norgan.net>
@norgan
norgan force-pushed the feat/discovery-progress-status branch from 494d826 to b968289 Compare October 18, 2026 04:23
@norgan
norgan marked this pull request as ready for review October 18, 2026 04:24
@camilasan

Copy link
Copy Markdown
Member

@norgan we will need more information on what you are trying to achieve with this PR.
What exactly you are trying to solve?
You are talking about visibility and you are changing strings, do you have screenshots of the changes?

@camilasan camilasan self-assigned this Oct 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants