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

feat: burn tax split and tax exemption #1

Merged
merged 1 commit into from
Feb 21, 2023
Merged

feat: burn tax split and tax exemption #1

merged 1 commit into from
Feb 21, 2023

Conversation

inon-man
Copy link
Contributor

@inon-man inon-man commented Feb 21, 2023

Description

This PR is required for supporting burn tax split, and tax exemption list
However, I found that current repository has files for Terra 2.0 not the Terra Classic.

Changes

  • use hard copied proto files (from classic-terra/core, classic-terra/cosmos-sdk) instead of using submodule
  • rollback java, python to classic version
  • treasury: support tax split rate and tax exemption

* treasury: support tax split rate and tax exemptio
* rollback java, python to classic version
@inon-man inon-man changed the title Updates for 1.1.0 feat: burn split rate and tax exemption Feb 21, 2023
@inon-man inon-man added the enhancement New feature or request label Feb 21, 2023
@inon-man inon-man self-assigned this Feb 21, 2023
@inon-man inon-man changed the title feat: burn split rate and tax exemption feat: burn tax split and tax exemption Feb 21, 2023
Copy link

@ZaradarBH ZaradarBH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Java, Python and JS in one big ball of fun. I think its clear why we call you superman ;) Great work buddy, LGTM :)

Copy link
Contributor

@Man12C Man12C left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work Inon-Man!

@ZaradarBH ZaradarBH merged commit 0d571cd into main Feb 21, 2023
@ZaradarBH ZaradarBH added the in scope Issues that are part of the work approved by the community label Feb 21, 2023
@ZaradarBH ZaradarBH added this to the Terrad - v1.1.0 milestone Feb 21, 2023
@inon-man inon-man deleted the tax-exemption branch July 9, 2023 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request in scope Issues that are part of the work approved by the community
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants