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

feat: add simulation on liquidstaking, improve logics #192

Merged
merged 14 commits into from
Feb 17, 2022
Merged

feat: add simulation on liquidstaking, improve logics #192

merged 14 commits into from
Feb 17, 2022

Conversation

dongsam
Copy link
Contributor

@dongsam dongsam commented Feb 15, 2022 •
edited
Loading

Description

closes: #91

Tasks

  • feat: improve liquid unstaking logic to cover no total liquid token case
  • feat: add voting_power endpoint
  • fix: event of liquid stake, unstake for specify delegator
  • feat: add inflation endpoint for mint and fix mint simultion, genesistime
  • fix: mint inflation schedule on params
  • feat: add voting_power endpoint
  • fix: apply genTime for mint gensim
  • test: add voting power unit test and tally simulation

References

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Appropriate labels applied
  • Targeted PR against correct branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Re-reviewed Files changed in the Github PR explorer
  • Review Codecov Report in the comment section below once CI passes

@dongsam dongsam force-pushed the 91-add-simulation-on-liquidstaking branch from 9031f94 to dd33b61 Compare February 15, 2022 13:43
@dongsam
feat: add voting_power endpoint
dd05388
@dongsam
fix: integration test for mint params
3d6f93b
@dongsam
fix: apply genTime for mint gensim
06ace99
@dongsam
Merge branch '91-add-simulation-on-liquidstaking' into 91-add-simulat…
aa648ce 
…ion-on-liquidstaking-wip

* 91-add-simulation-on-liquidstaking:
  fix: apply genTime for mint gensim
  fix: integration test for mint params
  feat: add voting_power endpoint
  fix: mint inflation schedule on params

# Conflicts:
#	x/liquidstaking/simulation/proposals.go
@dongsam
test: add voting power unit test and tally simulation
aced90b
@dongsam
fix: tidy simulations, add validator voting power
6923f89
@dongsam dongsam force-pushed the 91-add-simulation-on-liquidstaking branch from 72cd4bd to 6923f89 Compare February 17, 2022 10:47
@dongsam dongsam self-assigned this Feb 17, 2022
@dongsam dongsam added this to the v1.0 milestone Feb 17, 2022
@dongsam dongsam marked this pull request as ready for review February 17, 2022 11:04
@dongsam dongsam merged commit 3fa5416 into main Feb 17, 2022
@dongsam dongsam linked an issue Feb 22, 2022 that may be closed by this pull request
add simulation, test, query on mint module #109
Closed
4 tasks
@dongsam dongsam mentioned this pull request Feb 22, 2022
Closed
4 tasks
@hallazzang hallazzang deleted the 91-add-simulation-on-liquidstaking branch February 28, 2022 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jaybxyz jaybxyz Awaiting requested review from jaybxyz

@hallazzang hallazzang Awaiting requested review from hallazzang hallazzang is a code owner

Assignees

@dongsam dongsam

Labels
dependencies state breaking x/liquidstaking x/mint
Projects
None yet
Milestone
v1.0
Development

Successfully merging this pull request may close these issues.

add simulation, test, query on mint module Add simulation on liquidstaking
1 participant
@dongsam