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.

Sign up for GitHub

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

Jump to bottom

Supply Module #4210

Closed
wants to merge 48 commits into from
Closed

Supply Module #4210

wants to merge 48 commits into from

Conversation

fedekunze
Copy link
Collaborator

@fedekunze fedekunze commented Apr 26, 2019 •
edited
Loading

Closes #3972

  • Targeted PR against correct branch (see CONTRIBUTING.md)

  • 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: sdkch 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)

fedekunze added 30 commits April 10, 2019 19:53
@fedekunze
supply and token holder
cf8a1c9
@fedekunze
move logic to bank module
f36e8a8
@fedekunze
token minter
e2c5216
@fedekunze
fix interfaces
c173225
@fedekunze
supplier, bank keeper and update minting logic
d18b617
@fedekunze
remove token minter as per Alessio's comments
a8b2f9c
@fedekunze
minor fixes to supply and keeper interfaces
f8112f3
@fedekunze
genesis, add codec and cleanup
8dc2378
@fedekunze
rename pool supply; genesis supply
908d78b
@fedekunze
update bank keeper and holders
5c1cd32
@fedekunze
use token holder on gov instead of acc
7a3787c
@fedekunze
minor tests instances fixes
949c342
@fedekunze
module supply
28b17e5
@fedekunze
more cleanup
3a5eed9
@fedekunze
errors and more cleanup
575e484
@fedekunze
genesis stuff
053bb86
@fedekunze
fixes; now it builds
b62f9b2
@fedekunze
fix total supply
4e0d389
@fedekunze
delete token holders
390d0bf
@fedekunze
implement module accounts
62d8161
@fedekunze
check for module acc on genesis
044a0c6
@fedekunze
update vesting and circulating supply
913793c
@fedekunze
move invariants out of staking
c2b30ce
@fedekunze
update gov genesis
4fcf036
@fedekunze
update invariants
34a658a
@fedekunze
more invariance and total supply
d2c271e
@fedekunze
add rewards to total supply
a22bcca
@fedekunze
add remaining
b93fc11
@fedekunze
total supply invariant
021e6c9
@fedekunze
changes from develop
6111451
@fedekunze fedekunze added the WIP label Apr 26, 2019
@fedekunze fedekunze mentioned this pull request May 2, 2019
Closed
5 tasks
@fedekunze
Copy link
Collaborator Author

split and moved to #4255 to rely only on keepers and queriers

@fedekunze fedekunze closed this May 2, 2019
@fedekunze fedekunze deleted the fedekunze/3972-bank-total-supply branch May 2, 2019 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Total Supply Feature
1 participant
@fedekunze