search

breadwallet / breadwallet-android

The Android version of the Bread bitcoin wallet.
MIT License
389 stars 424 forks source link

Reasons behind the `Error: Could not publish transaction.` #151

Closed dabitdev closed 4 years ago

dabitdev commented 4 years ago

Hi, sometimes I am getting the following error sending BTC to another wallet:

Screen Shot 2020-07-13 at 12 30 04 PM

Could you help me understand what are the possible reasons for this error? When I get this error if I keep retrying it is will keep failing, the only work around is to restart the app.

DrewCarlson commented 4 years ago

This is the default message if an error reason cannot be determined though in your case it's likely due to a bug. If you're still having frequent send issues in 4.4.1, please let us know.

dabitdev commented 4 years ago

Hi @DrewCarlson I am maintaining a fork (forked on 4.4.1), looks like the app can go to a state where keeps failing. At what layer is this error coming? (from core)? It be possible that the wallet has to be fully sync.

On the other hand, could I check a changelog or a more specific readme file to see all the changes / bugfixies to see how critical is to posponed a rebase to my fork.

Thank you

DrewCarlson commented 4 years ago

We do not currently maintain a change log outside of the Play Store. That message is typically a result of calling a Core API but in this case it may be an issue in application code (see SendSheetHandler). If your wallet is in P2P mode (the fastsync setting is off), you'll need to wait until the sync is complete or this error is guaranteed.

dabitdev commented 4 years ago

PlayStore change log is not very useful for developers. Thank you, for you fast response.

Best, David