Update list_balances documentation
#421
Conversation
| @@ -1368,6 +1368,8 @@ impl Node { | |||
| } | |||
|
|
|||
| /// Retrieves an overview of all known balances. | |||
| /// | |||
| /// Note that to retrieve the latest on-chain balance, you need to call [`Node::sync_wallets`] first. | |||
There was a problem hiding this comment.
No, actually use of sync_wallets is more or less discouraged. Users should use the background syncing whenever possible.
There was a problem hiding this comment.
Noted. Do you think we should document that in the README? With a local experiment, sync_wallet was the only way I was able to get it to update the on-chain balance.
There was a problem hiding this comment.
Noted. Do you think we should document that in the README?
Mhh, so it is already noted on the sync_wallet docs. I guess that should be sufficient? It's surely not so important that it needs to go into the README?
With a local experiment, sync_wallet was the only way I was able to get it to update the on-chain balance.
Huh, I'd be interested to learn more about that. You should get the update eventually, on Esplora the default onchain_wallet_sync_interval_secs is 80 seconds to avoid hitting rate-limiting. So you should see updates after that the latest, or can of course try to reduce the interval as long as you don't see a lot of HTTP 429 errors popping up.
No description provided.