DeFiCh / ain

DeFi Blockchain - enabling decentralized finance on Bitcoin
https://defichain.com
MIT License
408 stars 122 forks source link

Neutral vote is mistakenly treated the same as a no-vote for on-chain gov #1704

Closed uzyn closed 1 year ago

uzyn commented 1 year ago

Caution

As on-chain governance is activated soon and likely before this consensus error is able to be fixed, do not cast neutral votes before this bug is addressed or it would be counted as a no-vote.

Bug

What happened:

Neutral votes for CFP and VoC are counted the same as no-votes, it should not be that way.

Reference: https://github.com/DeFiCh/ain/blob/0989ff6fde4530a83fa2c3649c2642615e36714d/src/masternodes/validation.cpp#L2235-L2238

Example:

What you expected to happen:

Neutral votes for CFP and VoC should not be counted as no-votes.

Example:

defichain-bot commented 1 year ago

@uzyn: Thanks for opening an issue, it is currently awaiting triage.

The triage/accepted label can be added by foundation members by writing /triage accepted in a comment.

Details I am a bot created to help the [DeFiCh](https://github.com/DeFiCh) developers manage community feedback and contributions. You can check out my [manifest file](https://github.com/DeFiCh/ain/blob/master/.github/governance.yml) to understand my behavior and what I can do. If you want to use this for your project, you can check out the [DeFiCh/oss-governance-bot](https://github.com/DeFiCh/oss-governance-bot) repository.