core icon indicating copy to clipboard operation
core copied to clipboard

Rework of tax to be included in gas

Open StrathCole opened this issue 1 year ago • 0 comments

Summary of changes

  • gas prices are added as on-chain governance parameters
  • tax is converted to gas by using gas price
  • ante handler only takes few gas, so on failed txs no tax is paid
  • overpaid tax is not deducted (depending on amount of fees sent)
  • tax for any denom can be paid in LUNC

Implementation draft for proposal #11873

Report of required housekeeping

  • [ ] Github issue OR spec proposal link
  • [x] Wrote tests
  • [x] Updated API documentation (client/lcd/swagger-ui/swagger.yaml)
  • [ ] Added a relevant changelog entry: clog add [section] [stanza] [message]

(FOR ADMIN) Before merging

  • [ ] Added appropriate labels to PR
  • [ ] Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
  • [ ] Confirm added tests are consistent with the intended behavior of changes
  • [ ] Ensure all tests pass

StrathCole avatar Nov 20 '23 10:11 StrathCole