Electric-Coin-Company / zcash-android-wallet-sdk

Native Android SDK for Zcash
MIT License
5 stars 10 forks source link

Use more precise error handling in `refreshWalletSummary` #1374

Open nuttycom opened 9 months ago

nuttycom commented 9 months ago

At present, receiving an error from getWalletSummary and receiving the None result are conflated; a recurrent error could make the wallet appear to be "stuck at zero" and while the existing logging in getWalletSummary will record the error, it likely needs to propagate upward to somewhere that it can be handled more gracefully and potentially be reported to the user.

_Originally posted by @nuttycom in https://github.com/Electric-Coin-Company/zcash-android-wallet-sdk/pull/1372#discussion_r1470399754_