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

R4R: Community pool spend proposal #4329

Merged
merged 17 commits into from
May 21, 2019
Merged

R4R: Community pool spend proposal #4329

merged 17 commits into from
May 21, 2019

Conversation

cwgoes
Copy link
Contributor

@cwgoes cwgoes commented May 13, 2019

Implement the "CommunityPoolSpendProposal" as described in Cosmos Hub proposal 7.

Blocked on passage of proposal 7 - edit: passed.
Probably blocked on #4159 - edit: merged.

Also a useful test of Git flow for merging features passed in governance proposals.

  • Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
  • Wrote tests
  • Updated relevant documentation (docs/)
  • Added a relevant changelog entry: clog add [section] [stanza] [message]
  • rereviewed Files changed in the github PR explorer

For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@cwgoes cwgoes added C:x/distribution distribution module related feature C:x/gov labels May 13, 2019
@codecov
Copy link

codecov bot commented May 13, 2019

Codecov Report

Merging #4329 into master will decrease coverage by 0.87%.
The diff coverage is 20%.

@@            Coverage Diff            @@
##           master   #4329      +/-   ##
=========================================
- Coverage   58.08%   57.2%   -0.88%     
=========================================
  Files         235     241       +6     
  Lines       14881   15125     +244     
=========================================
+ Hits         8643    8653      +10     
- Misses       5608    5842     +234     
  Partials      630     630

@cwgoes cwgoes mentioned this pull request May 13, 2019
13 tasks
@cwgoes cwgoes marked this pull request as ready for review May 20, 2019 13:44
@cwgoes cwgoes changed the title Community pool spend proposal R4R: Community pool spend proposal May 20, 2019
Copy link
Contributor

@alexanderbez alexanderbez left a comment

Choose a reason for hiding this comment

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

Great job @cwgoes, looks very clean. I left a few minor bits of feedback 👌

simapp/app.go Show resolved Hide resolved
simapp/sim_test.go Show resolved Hide resolved
x/bank/simulation/msgs.go Show resolved Hide resolved
x/distribution/client/cli/tx.go Outdated Show resolved Hide resolved
x/distribution/client/rest/rest.go Outdated Show resolved Hide resolved
x/distribution/keeper/proposal_handler.go Show resolved Hide resolved
x/distribution/keeper/proposal_handler.go Show resolved Hide resolved
x/distribution/types/proposal.go Show resolved Hide resolved
x/gov/simulation/msgs.go Show resolved Hide resolved
Copy link
Contributor

@alexanderbez alexanderbez left a comment

Choose a reason for hiding this comment

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

ACK. Thanks @cwgoes. Can you create a corresponding PR in Gaia, which just points to this branch for now?

@cwgoes
Copy link
Contributor Author

cwgoes commented May 20, 2019

ACK. Thanks @cwgoes. Can you create a corresponding PR in Gaia, which just points to this branch for now?

cosmos/gaia#10

Copy link
Contributor

@alessio alessio left a comment

Choose a reason for hiding this comment

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

Just a tiny package reorg - REST types should live inside the module's rest package

x/distribution/client/utils/utils.go Outdated Show resolved Hide resolved
@alessio alessio merged commit dd89c32 into cosmos:master May 21, 2019
alessio pushed a commit to cosmos/gaia that referenced this pull request May 21, 2019
bchainexpt pushed a commit to bchainexpt/Blockchain-wallet-andorid-app that referenced this pull request Apr 4, 2022
larry0x pushed a commit to larry0x/cosmos-sdk that referenced this pull request May 22, 2023
Implement the "CommunityPoolSpendProposal" as described in Cosmos Hub proposal 7.

Also a useful test of Git flow for merging features passed in governance proposals.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:x/distribution distribution module related C:x/gov
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants