Skip to content

Conversation

@ajey35
Copy link

@ajey35 ajey35 commented Jan 22, 2026

Disabled Auto-Sync on Wallet Page

This PR fixes #58 that the wallet page does not auto-sync and only loads data from cache unless the user manually refreshes.

Verified Changes

  • updateBalance, updateTransactions, updateUtxos now return early when useCache === true.
  • Wallet initialization uses cache-only loading.
  • Manual refresh still triggers network fetches (useCache === false).

Behavior

  • First visit (no cache): Empty/zero values, no network calls.
  • Cached visit: Instant load from cache, no network calls.
  • Refresh click: Network calls execute and cache updates.

File Changed

  • src/components/wallet/Wallet.js

Copy link
Collaborator

@keraliss keraliss left a comment

Choose a reason for hiding this comment

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

You're disabling the auto sync totally, which is not the ideal ux.
The issue is that users expect to see updated data when:

  • They first load the wallet page
  • They send funds
  • They receive funds
    So we need a nuanced approach. we need to think of a way of triggering syncs when meaningful changes happen, like initial load, after tx, but stop the spontaneous triggers which is lagging the ui.
    Simply removing all auto-sync means users see stale data ,or 0 balance on fresh load until they manually click refresh, which breaks expected behavior.we need to identify what is triggering the excessive sync and stop that.

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.

Don't auto sync on wallet page

2 participants