feat!: upstream expedited proposals #14720
Conversation
julienrbrt
changed the title
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| // EXCEPT when an expedited proposal fails. | ||
| if !(proposal.Expedited && !passes) { | ||
| if burnDeposits { | ||
| keeper.DeleteAndBurnDeposits(ctx, proposal.Id) |
Check warning
Code scanning / CodeQL
Panic in BeginBock or EndBlock consensus methods
| if burnDeposits { | ||
| keeper.DeleteAndBurnDeposits(ctx, proposal.Id) | ||
| } else { | ||
| keeper.RefundAndDeleteDeposits(ctx, proposal.Id) |
Check warning
Code scanning / CodeQL
Panic in BeginBock or EndBlock consensus methods
| // according to the regular proposal rules. | ||
| proposal.Expedited = false | ||
| params := keeper.GetParams(ctx) | ||
| endTime := proposal.VotingStartTime.Add(*params.VotingPeriod) |
Check warning
Code scanning / CodeQL
Panic in BeginBock or EndBlock consensus methods
| endTime := proposal.VotingStartTime.Add(*params.VotingPeriod) | ||
| proposal.VotingEndTime = &endTime | ||
|
|
||
| keeper.InsertActiveProposalQueue(ctx, proposal.Id, *proposal.VotingEndTime) |
Check warning
Code scanning / CodeQL
Panic in BeginBock or EndBlock consensus methods
|
@julienrbrt One of the things Osmosis also implemented was a higher deposit amount for expedited proposals. This, paired with the minimum initial deposit ratio, would help remedy spam proposals. We (KYVE) would love to see this included 🙌🏻 |
julienrbrt
force-pushed
the
julien/expedited-proposal-upstream
branch
from
6ea123b
to
d3dc533
Compare
x/gov/abci_test.go
Outdated
| @@ -1,3 +1,5 @@ | |||
| // TODO update this file | |||
There was a problem hiding this comment.
Upstreaming the tests here.
| } | ||
|
|
||
| if passes { | ||
| keeper.RemoveFromActiveProposalQueue(ctx, proposal.Id, *proposal.VotingEndTime) |
Check warning
Code scanning / CodeQL
Panic in BeginBock or EndBlock consensus methods
Minimal backport of cosmos#14720
Co-authored-by: Julien Robert <[email protected]>
Description
Closes: #14694
As per #14720 (comment), up streaming osmosis-labs@8b021d2 as well
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
!to the type prefix if API or client breaking changeCHANGELOG.mdReviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...
!in the type prefix if API or client breaking change