docs: update proposal and params for release v1.1.0

[nghuyenthevinh2000]

I. Burn tax split to community pool

Summary

Previously, the community utilized the seigniorage RewardWeight parameter to remint a portion of the burned supply to the community pool. This was confusing to many people in the community. The third governance proposal is seeking consensus around the creation of a new parameter that does not utilize the seigniorage remint policy, but has an independent ability to directly send a percentage of the on-chain tax to the community pool. The initialization of this parameter will be 10% community pool, 90% burn as noted in governance proposal #11111, which was the consensus before the Binance requests to turn off seigniorage (and are being alternatively handled by feature #2).

Specs

• AnteHandler: https://github.com/classic-terra/core/blob/main/custom/auth/ante/burntax.go
• BurnTaxSplit parameter: https://github.com/classic-terra/core/blob/main/x/treasury/types/params.go#L23
• How to change BurnTaxSplit through gov? check out this script that does it: test: add burn tax split gov script test #126

Tests

• TestSplitTax:

  • It will go through several burn tax split rate called %split_rate
  • It will deduct tax from user to FeeCollector (first check to see if FeeDecorator has the correct tax amount)
  • It will send the transaction to BurnTaxFeeDecorator
  • It will check if Community Pool has the amount of tax*%split_rate
  • It will check if Total Supply has less amount of tax*(1-%split_rate) than that in the beginning (since tax*(1-%split_rate) is burned)
  • It will check if FeeCollector has zero amount (since 100% tax is sent to Community pool and Burn)

• BurnTaxSplit param gov change test:

  • It will setup a node
  • It will submit a param gov to change BurnTaxSplit
  • It will output BurnTaxSplit before and after gov

• Upgrade check: Make sure that state migration in x/treasury module works

II. Burn tax exemption list

Summary

Binance has requested that the internal movement between Binance owned wallets be exempt from the burn tax (currently set to 0.2%). A list of Binance owned wallets has been provided to the community via the above announcement. It was confirmed with Binance that movement to and from Binance wallets will still be taxed. This means deposits and withdrawals are not exempt, but rather only the internal movements between wallets will be exempt for their internal security purposes. This first governance proposal is seeking consensus around this request from Binance. The implementation of this whitelist feature will be parameterized, with the initial list set to the provided Binance list. Thus, if this feature is accepted to be adopted, future changes to this whitelist can be controlled by the community in a governance parameter change proposal.

Specs

• AddBurnTaxExemptionAddressProposal
• RemoveBurnTaxExemptionAddressProposal
• How to submit proposal through gov? check out this script that does it: https://github.com/classic-terra/core/blob/main/scripts/burn-tax-exemption-test.sh

Tests

• TestFilterRecipient:

  • Create Msgs for various case:
    • MsgSend(binance -> binance)
    • MsgSend(normal -> normal)
    • MsgSend(binance -> normal), MsgSend(binance -> binance)
    • MsgSend(binance -> binance), MsgMultiSend(binance -> normal, binance -> binance)
  • Send Msg to BurnTaxFeeDecorator
  • Check amount burn before and after
  • Check community pool before and after
  • Check FeeCollector before and after

• Tax Exemption List gov change:

  • Create a node
  • Make a gov to add addresses to Tax Exemption List
  • Check if tax exemption list has those addresses

• Upgrade check: Make sure that state migration in x/treasury module works and Binance addresses are added to Tax Exemption List

[nghuyenthevinh2000]

@edk208 @inon-man @ZaradarBH @fragwuerdig

Can you have a look and comment on this one?

[edk208]

beautiful thank you