search

elesto-dao / elesto

The ElestoDAO node
Apache License 2.0
4 stars 1 forks source link

fix(mint): fix pruning panic, add developer and community funding #161

Closed gsora closed 2 years ago

gsora commented 2 years ago

Pretty much what the title says.

Before running this PR:

  1. all nodes state must be rolled back to height 1151280
  2. all nodes must go back to node version v1.0.0-rc2 with pruning in any mode except nothing (we wanna have pruning!)
  3. a new proposal for upgrade testnetUpgrade20220706 must be created and approved

The fix for the pruning issue has been taken from https://github.com/cosmos/ibc-go/issues/1088.

Please, review the issue as well as this PR before giving a ✅!

Thanks <3

PSA

This PR previously hardcoded utsp as the default MintDenom for the x/mint module params, but after a conversation with @noandrea we decided it's best we leave it as sdk.DefaultBondDenom, and let the chain admin set denoms via genesis.json editing.

The 2022-07-06 migration will still set MintDenom to utsp, but we do that to try having a working SDK migration in place.

167 will take care of generalizing tests instead of testing for "stake" everywhere, so this PR must be merged first.

Edit

167 has been approved and merged here instead, so this PR now enables developer and community funding as well.