-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
remove treasury from Encointer runtime #176
Comments
How did #158 cause this? Yea if the treasury is not used here then just remove it i guess. |
Why only << 1KSM: because almost all extrinsics are paid in community currency (thus purchasing power adjusted). These CC fees are burned. Only registering new communities needs KSM which went to treasury #158 removed our custom deal-with-fees and replaced it with the standard systemchain impl. TBH I haven't tested the behaviour of the new setup in all cases yet |
…tead of treasury (#270) closes #183 closes #176 closes #146 (should be closed already, but this PR adds more) This PR aligns collator selection and fee distribution logic of Encointer with other system chains and thereby is a small step towards [self-sustainability of infrastructure](https://forum.polkadot.network/t/towards-encointer-self-sustainability/4195) * [x] remove treasury (and leave negligible funds on trsry account) * [x] add collator-selection, session and authorship pallets * [x] instead of burning fees in community currency , send them to block authors like native KSM * [x] remove DMP queue pallet (pure cleanup, no change in functionality) * [x] add migration to inject existing collators as invulnerables * [x] remove applied migrations from release 1.2.0 steps to reproduce the new fee logic: https://gist.github.com/brenzi/ca9e0c326198c73ff2dd0a2e74468053 --------- Co-authored-by: joe petrowski <[email protected]>
Since #158, Encointer's treasury will no longer receive fees. While this is more the result of an oversight AFAIU, it may actually make sense to get rid of the treasury on the Encointer system chain altogether.
The new multi-asset treasury is way too complex for just collecting KSM fees. Yet, it is not powerful enough to be used for our plans of per-community treasuries
I suggest to simply remove the treasury pallet and ignore the negligible funds currently in the treasury (<<1KSM)
The text was updated successfully, but these errors were encountered: