Skip to content
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

[test] Treasury surplus burns in absence of proposals #3402

Closed
wants to merge 1 commit into from

Conversation

ekovalev
Copy link
Member

A test that demonstrates that in case no proposals have been registered with Treasury, the treasury account positive balance is halved every SpendPeriod blocks (1 day worth of blocks in Vara runtime) thereby causing the total issuance reduction. Unless specifically handled - that is, a BurnDestination is provided that takes care of the resulting NegativeImbalance.
Apparently, the rationale behind this is that Treasury is not supposed to store funds, effectively reducing the amount in circulation. It's only when a proposal is created, funds can flow in the Treasury until enough thereof is accumulated.

It means that if the idea is to be able to store funds in Treasury for some future purpose, we need to funnel the burned amount back to the Treasury itself.

@ekovalev ekovalev added the A5-dontmerge PR should not be merged yet label Oct 11, 2023
@ekovalev ekovalev force-pushed the ek-treasury-surplus-burns branch from 2303940 to 88776ec Compare October 11, 2023 14:26
@ekovalev ekovalev force-pushed the ek-treasury-surplus-burns branch from 88776ec to 33cba01 Compare October 11, 2023 14:27
@ekovalev ekovalev closed this Oct 12, 2023
@ukint-vs ukint-vs deleted the ek-treasury-surplus-burns branch November 13, 2023 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A5-dontmerge PR should not be merged yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant