-
Notifications
You must be signed in to change notification settings - Fork 851
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
impr(inflation
): Rename total supply endpoint
#536
Conversation
ENG-207 Rename /evmos/inflation/v1/total_supply -> /evmos/inflation/v1/circulating_supply
This is a breaking change so we will need to inform partners that this will change. This clarifies what the difference is between querying total supply on |
Codecov Report
@@ Coverage Diff @@
## main #536 +/- ##
=======================================
Coverage 81.56% 81.56%
=======================================
Files 115 115
Lines 6375 6375
=======================================
Hits 5200 5200
Misses 1018 1018
Partials 157 157
|
inflation
): Rename total supply endpoint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Why are so many files being autogenerated?
This happened when running |
what version of |
* loredana/ENG-214-add-epoch-types: (40 commits) Update evmos v3 -> v4 after latest changes Add year and hour epochs Update evmos v3 -> v4 after latest changes uncomment check feat: Inbound / outbound peers & default seeds (evmos#541) upgrade: Update Evmos go.mod version `v3` -> `v4` (evmos#557) Add change log Add migration logic for epoch change Fix epochs unit tests Revert to ascending order Store epoch information by duration and by identifier audit(erc20): add types tests and update comments (evmos#550) [ENG-219] bump erc20 test coverage (evmos#546) audit(erc20): state machine audit (evmos#547) audit(erc20): Changes from api audit (evmos#544) imp: update default min-gas-prices (evmos#543) imp: use constants for epochs IDs (evmos#539) fix: upgrade client router key (evmos#537) impr(`inflation`): Rename total supply endpoint (evmos#536) fix: `buf protoc` was moved to `buf alpha protoc` (evmos#462) ...
Description
This PR renames the endpoint
/evmos/inflation/v1/total_supply
to/evmos/inflation/v1/circulating_supply
to avoid confusion with thetotalSupply
from the bank module. The circulating supply excludes the team vesting allocation in the first year.Note that this is a state machine breaking change.