-
Notifications
You must be signed in to change notification settings - Fork 838
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
audit(erc20): Changes from api audit #544
Conversation
Codecov Report
@@ Coverage Diff @@
## main #544 +/- ##
==========================================
+ Coverage 81.56% 81.64% +0.07%
==========================================
Files 115 115
Lines 6375 6275 -100
==========================================
- Hits 5200 5123 -77
+ Misses 1018 997 -21
+ Partials 157 155 -2
|
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! Great work 🚀 Can you add a changelog entry? 🙏
* 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 adds improvements from performing an API audit on the
x/erc20
module:updateTokenPairERC20Proposal
functionalityrelay
toconversion
Note that this change is API breaking due to the endpoint renaming.