Skip to content

Feature: show sync pipeline as left-side checklist during sync#123

Merged
TheRealSavi merged 3 commits into
TheRealSavi:mainfrom
Ralim:ralim/sync-pipeline-indicator
Jun 15, 2026
Merged

Feature: show sync pipeline as left-side checklist during sync#123
TheRealSavi merged 3 commits into
TheRealSavi:mainfrom
Ralim:ralim/sync-pipeline-indicator

Conversation

@Ralim

@Ralim Ralim commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

This was my first pass for adding a left side indication panel during sync to show progress.

Let me know anything else I can do to help / also feel free to re-write or change it up however you want (:

I don't love having DEFAULT_PIPELINE but I didn't spot a good gui layer to put this in otherwise as the current progress view wasn't aware of the stages; and the SyncEngine objects rapidly got harder to follow.

Closes #121

EDIT: I'll re-push later to fixup CI I didn't run locally.
EDIT: CI updated & rebased

@Ralim Ralim force-pushed the ralim/sync-pipeline-indicator branch from 860788e to f988b40 Compare June 15, 2026 02:50
@Ralim Ralim force-pushed the ralim/sync-pipeline-indicator branch from f988b40 to 6e2ac4f Compare June 15, 2026 02:55
@TheRealSavi TheRealSavi merged commit a4a1d1a into TheRealSavi:main Jun 15, 2026
1 check passed

@TheRealSavi TheRealSavi left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

no worries on implementation details - as the syncengine truely is a big mess. I have some changes on my machine that will be coming to live soon that will refactor it to be more simple.

@Ralim

Ralim commented Jun 15, 2026

Copy link
Copy Markdown
Contributor Author

Your doing amazing work, I'm so appreciative of you doing this as it does exactly what I wanted. In happy I can help a little :)

On one hand the syncing is complex, on the other hand it is the main backbone of the program so of anything was to be complex, it's the bit I'd expect.

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.

[Feature] Clear sync progress states

2 participants