-
Notifications
You must be signed in to change notification settings - Fork 608
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
* feat(spike): taker fee (#6034) * add taker fee determination and extraction * remove osmo multi hop logic * route to both community pool and staking rewards * fix a handfull of tests * assign taker fee to pool at time of creation * pull taker fee direct from pool struct * fix more tests * fix tests, set back up osmo multi hop discount * correct taker fee extraction * regen mocks * abstract taker fee to pool manager (highest lvl) * fix extract cmd * update changelog * tidy * remove param that no longer exists * add extra params to genesis logic * add back osmo multihop tests * add comment * fix e2e keeping prints * fixes * more test fixes * remove prints * more naive approach to determining taker fee * fix e2e * re-enable disabled test * minor cleaning * simplify params * not nullable * clean up * add whitelist set message denom pair taker fees * use real addresses * add gov prop for denom pair taker fee update * clean * move logic to its own taker_fee.go file * add CLI for gov prop for denomPairTakerFee * add admin address denomPairTakerFee cli msg * clean and simplifications * use authorized quote denoms from poolmanger * remove stableswap taker fee * sim msg change * add test for CLI * msg server tests * add route_test test * add gov_test.go * add msgs_test.go * remove print line * change from v18 to v19 * Update upgrades.go * set default taker fee to zero in upgrade handler * conflicts * Update proto/osmosis/poolmanager/v1beta1/genesis.proto * Update proto/osmosis/poolmanager/v1beta1/genesis.proto * Update x/poolmanager/taker_fee.go * Generated protofile changes * rename extractTakerFeeAndDistribute to chargeTakerFee * clean up * comment * rename * godoc for NonNativeFeeCollectorForCommunityPoolName * baseDenom to defaultFeesDenom name change * fix upgrade handler * fix AfterEpochEnd order of denoms when getting pools; reduce code dupl; tests * fix track volume * add key separator for pool manager * add comment to denom pair route * add basic lexicographical key test * update chargeTakerFee godoc * set the default taker fee back to non zero for e2e * change PoolMangerGetParams API to GetAuthorizedQuoteDenoms (same for setters) * update txfees & poolmanager READMEs with takerFee * fix merge main * add comment to denom pair taker fee * de-dup taker fee param validation * Update x/poolmanager/types/msgs.go * Apply suggestions from code review * Update x/poolmanager/taker_fee.go * remove unneeded setup test * get pool creation free from previous * take non native out of name * undo param pull instead of default * use current pool creation fee * use default * e2e --------- Co-authored-by: devbot-wizard <[email protected]> Co-authored-by: Roman <[email protected]> Co-authored-by: github-actions <[email protected]> Co-authored-by: alpo <[email protected]> Co-authored-by: Dev Ojha <[email protected]> Co-authored-by: Dev Ojha <[email protected]> (cherry picked from commit 5c8fd80) # Conflicts: # CHANGELOG.md # tests/e2e/e2e_test.go # x/poolmanager/export_test.go # x/poolmanager/router.go # x/poolmanager/types/keys.go * fix merge conflicts * change json * inc num heights to wait * Update .github/workflows/test.yml * test with v18 init image * removed old upgrade logic --------- Co-authored-by: Adam Tucker <[email protected]> Co-authored-by: Adam Tucker <[email protected]> Co-authored-by: Dev Ojha <[email protected]>
- Loading branch information
1 parent
f992a7d
commit 908b279
Showing
62 changed files
with
2,626 additions
and
388 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.