[bisq-network/bisq] Vote Reveal - Silent Fail (#3758)

niyid notifications at github.com
Fri Dec 6 15:29:50 UTC 2019


<!-- 
- make yourself familiar with the CONTRIBUTING.md if you have not already (https://github.com/bisq-network/bisq/blob/master/CONTRIBUTING.md)
- make sure you follow our [coding style guidelines][https://github.com/bisq-network/style/issues)
- pick a descriptive title
- provide some meaningful PR description below
- create the PR
- in case you receive a "Change request" and/or a NACK, please react within 30 days. If not, we will close your PR and it can not be up for compensation.
- After addressing the change request, __please re-request a review!__ Otherwise we might miss your PR as we tend to only look at pull requests tagged with a "review required".
-->

Fixes #3408

The log below shows where the vote reveal failed silently as a result of insufficient wallet balance.
```
Oct-15 06:43:21.835 [JavaFX Application Thread] INFO  org.bitcoinj.wallet.Wallet: Completing send tx with 1 outputs totalling 0.00000596 BTC and a fee of 0.00 BTC/kB 
Oct-15 06:43:21.837 [JavaFX Application Thread] WARN  org.bitcoinj.wallet.Wallet: Insufficient value in wallet for send: needed 0.00001606 BTC more 
Oct-15 06:43:21.845 [JavaFX Application Thread] ERROR b.c.d.g.v.VoteRevealService: VoteRevealException{
     voteRevealTx=null,
     blindVoteTxId='3cf435e275b8e1518ea969fe90e0504bbed1d68135310b3c24546244fca90b5d',
     myVote=null
} bisq.core.dao.governance.votereveal.VoteRevealException: Exception at calling revealVote. 
Oct-15 06:43:24.448 [JavaFX Application Thread] INFO  b.c.d.m.DaoStateMonitoringService: Passed checkpoint Checkpoint {
     height=586920,
     hash=523aaad4e760f6ac6196fec1b3ec9a2f42e5b272
} 
```

You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bisq/pull/3758

-- Commit Summary --

  * Added interface and corresponding implementation to handle display of

-- File Changes --

    M core/src/main/java/bisq/core/dao/governance/votereveal/VoteRevealService.java (12)
    M core/src/main/resources/i18n/displayStrings.properties (3)
    M desktop/src/main/java/bisq/desktop/main/dao/DaoView.java (6)

-- Patch Links --

https://github.com/bisq-network/bisq/pull/3758.patch
https://github.com/bisq-network/bisq/pull/3758.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/3758
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20191206/8503d8dc/attachment.html>


More information about the bisq-github mailing list