Skip to content

[Tokenomics] Adding mint_equals_burn_claim_distribution and revamping tokenomics module#1618

Merged
Olshansk merged 67 commits intomainfrom
tokenomics_1607
Jul 16, 2025
Merged

[Tokenomics] Adding mint_equals_burn_claim_distribution and revamping tokenomics module#1618
Olshansk merged 67 commits intomainfrom
tokenomics_1607

Conversation

@Olshansk
Copy link
Copy Markdown
Collaborator

@Olshansk Olshansk commented Jul 1, 2025

tl;dr Added mint_equals_burn_claim_distribution so supplier rewards will also go to the validator & service owner.


Note

The documentation is not final yet so do not review it


Other changes:

  • Took is as an opportunity to cleanup & revamp the tokenomics module
  • Reviewed TODOs along the way
  • Found lots of issues in the e2e test module, which led to changes in other places while debugging.
Screenshot 2025-07-10 at 8 21 05 PM

@Olshansk Olshansk self-assigned this Jul 1, 2025
@github-actions github-actions bot added the consensus-breaking IMPORTANT! If the PR with this tag is merged, next release WILL HAVE TO BE an upgrade. label Jul 1, 2025
@Olshansk Olshansk added tokenomics Token Economics - what else do you need? and removed consensus-breaking IMPORTANT! If the PR with this tag is merged, next release WILL HAVE TO BE an upgrade. labels Jul 1, 2025
@Olshansk Olshansk added this to Shannon Jul 1, 2025
@Olshansk Olshansk moved this from 📋 Backlog to 👀 In review in Shannon Jul 1, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Shannon Jul 1, 2025
@Olshansk Olshansk moved this from 👀 In review to 🏗 In progress in Shannon Jul 1, 2025
@Olshansk Olshansk marked this pull request as draft July 1, 2025 06:44
@github-actions github-actions bot added the consensus-breaking IMPORTANT! If the PR with this tag is merged, next release WILL HAVE TO BE an upgrade. label Jul 1, 2025
@Olshansk Olshansk changed the title [WIP][Tokenomics] Mint=Burn Updates First Iteration [WIP][Tokenomics] GlobalMint <> Mint=Burn Integration Jul 8, 2025
Copy link
Copy Markdown
Collaborator Author

@Olshansk Olshansk left a comment

Choose a reason for hiding this comment

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

About to submit a major overhaul before the first line of major changes.

Leaving this as a way to inspect the difference in github

@Olshansk Olshansk added protocol General core protocol related changes on-chain On-chain business logic labels Jul 9, 2025
@Olshansk Olshansk changed the title [WIP][Tokenomics] GlobalMint <> Mint=Burn Integration [WIP][Tokenomics] Adding mint_equals_burn_claim_distribution Jul 10, 2025
@Olshansk Olshansk marked this pull request as ready for review July 11, 2025 00:26
@Olshansk Olshansk changed the base branch from remove_pulsar to main July 11, 2025 18:21
@Olshansk Olshansk moved this from 🏗 In progress to 👀 In review in Shannon Jul 11, 2025
Copy link
Copy Markdown
Contributor

@red-0ne red-0ne left a comment

Choose a reason for hiding this comment

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

Great work! ⛰️

Submitting an initial review.

It's missing docs and some tests
image

@Olshansk Olshansk requested a review from red-0ne July 15, 2025 20:20
Copy link
Copy Markdown
Collaborator

@bryanchriswhite bryanchriswhite left a comment

Choose a reason for hiding this comment

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

image

Still looking at docs and TLM changes.

Olshansk and others added 5 commits July 16, 2025 11:41
Co-authored-by: Bryan White <bryanchriswhite@gmail.com>
Co-authored-by: Bryan White <bryanchriswhite@gmail.com>
## Summary

Update the dependency version for `github.com/pokt-network/smt` in the
`go.mod` file. The version has been upgraded from `v0.13.0` to
`v0.14.1`.

## Type of change

Select one or more from the following:

- [x] New feature, functionality or library
- [ ] Bug fix
- [ ] Code health or cleanup
- [ ] Documentation
- [ ] Other (specify)

## Sanity Checklist

- [x] I have updated the GitHub Issue Metadata: `assignees`,
`reviewers`, `labels`, `project`, `iteration` and `milestone`
- [ ] For docs: `make docusaurus_start`
- [ ] For small changes: `make go_develop_and_test` and `make test_e2e`
- [ ] For major changes: `devnet-test-e2e` label to run E2E tests in CI
- [ ] For migration changes: `make test_e2e_oneshot`
- [ ] 'TODO's, configurations and other docs
@Olshansk Olshansk requested a review from bryanchriswhite July 16, 2025 16:11
@Olshansk Olshansk merged commit 8270157 into main Jul 16, 2025
14 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Shannon Jul 16, 2025
@Olshansk Olshansk deleted the tokenomics_1607 branch July 16, 2025 20:17
bryanchriswhite added a commit that referenced this pull request Jul 17, 2025
* pokt/main:
  [Tokenomics] Adding `mint_equals_burn_claim_distribution` and revamping `tokenomics` module (#1618)
bryanchriswhite added a commit that referenced this pull request Jul 17, 2025
* issues/1497/event-fields2:
  [Tokenomics] Adding `mint_equals_burn_claim_distribution` and revamping `tokenomics` module (#1618)
  fix: backwards compatible field indices
  [Disk Utilization] refactor: remove unnecessary Msg.*Response fields (#1670)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consensus-breaking IMPORTANT! If the PR with this tag is merged, next release WILL HAVE TO BE an upgrade. on-chain On-chain business logic protocol General core protocol related changes tokenomics Token Economics - what else do you need?

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants