Skip to content

Add community tax removal to genesis json in scripts#285

Merged
Kbhat1 merged 1 commit into
masterfrom
kartik/AddCommunityTaxGenesis
Oct 6, 2022
Merged

Add community tax removal to genesis json in scripts#285
Kbhat1 merged 1 commit into
masterfrom
kartik/AddCommunityTaxGenesis

Conversation

@Kbhat1
Copy link
Copy Markdown
Contributor

@Kbhat1 Kbhat1 commented Oct 5, 2022

  • Adds a command to edit the genesis.json to set the community_tax param of the distribution module to 0
  • Tested end to end and verified community_tax param is 0

@Kbhat1 Kbhat1 requested a review from LCyson October 5, 2022 15:33
@Kbhat1 Kbhat1 merged commit 0079c45 into master Oct 6, 2022
masih pushed a commit that referenced this pull request Sep 29, 2025
@BrandonWeng
Always reset state between proposals (#285)
c6acf9c 
## Describe your changes and provide context
Similar changes to here:
cosmos/cosmos-sdk#15487

Our fork of cosmos-sdk has a custom integration to get ABCI++ working so
I'd like @codchen to take a second look
## Testing performed to validate your change
Unit tests
masih pushed a commit that referenced this pull request Sep 29, 2025
@BrandonWeng
Revert "Always reset state between proposals (#285)" (#292)
baf7640 
This reverts commit c6acf9c.

## Describe your changes and provide context
With optimistic processing this is not applicable on our side 

## Testing performed to validate your change
Launched a cluster with 3.0.3, with only this change applied and see
that it app hashes

![image](https://github.com/sei-protocol/sei-cosmos/assets/18161326/5d6b0606-aba5-4556-b2c9-6417572eee26)
masih pushed a commit that referenced this pull request Sep 30, 2025
@BrandonWeng @masih
Always reset state between proposals (#285)
c755d6c 
## Describe your changes and provide context
Similar changes to here:
cosmos/cosmos-sdk#15487

Our fork of cosmos-sdk has a custom integration to get ABCI++ working so
I'd like @codchen to take a second look
## Testing performed to validate your change
Unit tests
masih pushed a commit that referenced this pull request Sep 30, 2025
@BrandonWeng @masih
Revert "Always reset state between proposals (#285)" (#292)
b16c3a1 
This reverts commit c6acf9c.

## Describe your changes and provide context
With optimistic processing this is not applicable on our side 

## Testing performed to validate your change
Launched a cluster with 3.0.3, with only this change applied and see
that it app hashes

![image](https://github.com/sei-protocol/sei-cosmos/assets/18161326/5d6b0606-aba5-4556-b2c9-6417572eee26)
@masih masih deleted the kartik/AddCommunityTaxGenesis branch October 31, 2025 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@LCyson LCyson LCyson approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Kbhat1 @LCyson