Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync progress bar issues #2228

Closed
mrfelton opened this issue May 16, 2019 · 1 comment

Comments

1 participant
@mrfelton
Copy link
Member

commented May 16, 2019

Description

After opening a local wallet that needs some syncing to get current, the sync progress bar is not working correctly. It seems that we are incrementing the recovery height counter rather than the sync height counter.

Expected Behavior

Progress bar should work normally when re-syncing already synced wallet

Actual Behavior

Progress bar doesn't move

Possible Fix

Investigate why the recovery height counter is being incremented instead of the cfilter height counter.

Steps to Reproduce

Open a local wallet that has already synced but that needs some additional sync to get current

@mrfelton mrfelton added this to the v0.5.0-beta milestone May 16, 2019

@mrfelton mrfelton self-assigned this May 16, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue May 17, 2019

@mrfelton mrfelton changed the title Sync progress bar not working on resync of existing already synced wallet Sync progress bar issues May 17, 2019

mrfelton added a commit to mrfelton/zap-desktop that referenced this issue May 17, 2019

@mrfelton mrfelton referenced this issue May 17, 2019

Merged

fix(sync): improve sync progress bar reporting #2234

4 of 6 tasks complete
@mrfelton

This comment has been minimized.

Copy link
Member Author

commented May 18, 2019

Fixed via #2234

@mrfelton mrfelton closed this May 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.