Skip to content

Comments

Replace syncRatio with syncStatus#702

Merged
swansontec merged 1 commit intomasterfrom
william/sync-status
Feb 11, 2026
Merged

Replace syncRatio with syncStatus#702
swansontec merged 1 commit intomasterfrom
william/sync-status

Conversation

@swansontec
Copy link
Contributor

@swansontec swansontec commented Jan 31, 2026

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Description

none

Note

Medium Risk
Touches public types and redux plumbing for wallet syncing, so downstream apps/plugins may need updates and sync progress updates could regress if any path still assumes syncRatio.

Overview
Adds a richer wallet sync progress model by introducing EdgeSyncStatus (with totalRatio plus optional per-chain details) and exposing it as syncStatus on EdgeCurrencyWallet and EdgeMemoryWallet.

Plumbs a new engine callback onSyncStatusChanged through wallet callbacks, redux actions/reducer state (replacing the old CURRENCY_ENGINE_CHANGED_SYNC_RATIO flow), and API getters; keeps backwards compatibility by mapping deprecated onAddressesChecked and syncRatio to syncStatus.totalRatio. Also adds EdgeCurrencyInfo.syncDisplayPrecision for UI formatting guidance.

Written by Cursor Bugbot for commit e0ff8d2. This will update automatically on new commits. Configure here.


Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@swansontec swansontec merged commit c7fdb34 into master Feb 11, 2026
3 checks passed
@swansontec swansontec deleted the william/sync-status branch February 11, 2026 00:02
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