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.

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

Add Taker Fee Query #6680

Merged
merged 13 commits into from
Oct 13, 2023
Merged

Add Taker Fee Query #6680

merged 13 commits into from
Oct 13, 2023

Conversation

mattverse
Copy link
Member

Closes: #XXX

What is the purpose of the change

Adds taker fee query and add it to stargate whitelist.

Needed for necessary calculations for all contracts that have dependency on calculation.

Testing and Verifying

This change is a trivial rework / code cleanup without any test coverage.

Documentation and Release Note

  • Does this pull request introduce a new feature or user-facing behavior changes?
  • Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

  • Specification (x/{module}/README.md)
  • Osmosis documentation site
  • Code comments?
  • N/A

@mattverse
Copy link
Member Author

devbot help

@mattverse mattverse closed this Oct 12, 2023
@mattverse mattverse reopened this Oct 12, 2023
@czarcas7ic czarcas7ic closed this Oct 12, 2023
@czarcas7ic czarcas7ic reopened this Oct 12, 2023
Copy link
Member

@czarcas7ic czarcas7ic left a comment

Choose a reason for hiding this comment

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

Don't we need CLI for this as well as add to the query server? It seems this isn't registered rn

Would also feel good to run this on localosmosis if possible to prove the query works, after adding CLI and registering the message.

@mattverse
Copy link
Member Author

I didn't add CLI on purpose(didn't see any need for it), but on the second thought, why not :)
Added in the most recent commit and comfirmed that it fetches state correctly

./osmosisd q poolmanager trading-pair-taker-fee stake uosmo
taker_fee: "0.100000000000000000"

@mattverse mattverse closed this Oct 13, 2023
@mattverse mattverse reopened this Oct 13, 2023
@czarcas7ic czarcas7ic added the A:backport/v20.x backport patches to v20.x branch label Oct 13, 2023
@czarcas7ic czarcas7ic merged commit 144e088 into main Oct 13, 2023
1 check passed
@czarcas7ic czarcas7ic deleted the mattverse/takerfee-stargaate branch October 13, 2023 14:46
mergify bot pushed a commit that referenced this pull request Oct 13, 2023
* Add trader fee query

* Generated protofile changes

* Add Taker fee

* Add querier logic

* Generated protofile changes

* Add grpc guery.go

* Generated protofile changes

* Update proto/osmosis/poolmanager/v1beta1/query.proto

Co-authored-by: Adam Tucker <[email protected]>

* Update proto/osmosis/poolmanager/v1beta1/query.proto

Co-authored-by: Adam Tucker <[email protected]>

* Add cli

* Generated protofile changes

* register query in query server

---------

Co-authored-by: github-actions <[email protected]>
Co-authored-by: Adam Tucker <[email protected]>
Co-authored-by: Adam Tucker <[email protected]>
(cherry picked from commit 144e088)
czarcas7ic pushed a commit that referenced this pull request Oct 13, 2023
* Add trader fee query

* Generated protofile changes

* Add Taker fee

* Add querier logic

* Generated protofile changes

* Add grpc guery.go

* Generated protofile changes

* Update proto/osmosis/poolmanager/v1beta1/query.proto

Co-authored-by: Adam Tucker <[email protected]>

* Update proto/osmosis/poolmanager/v1beta1/query.proto

Co-authored-by: Adam Tucker <[email protected]>

* Add cli

* Generated protofile changes

* register query in query server

---------

Co-authored-by: github-actions <[email protected]>
Co-authored-by: Adam Tucker <[email protected]>
Co-authored-by: Adam Tucker <[email protected]>
(cherry picked from commit 144e088)

Co-authored-by: Matt, Park <[email protected]>
pysel pushed a commit that referenced this pull request Oct 17, 2023
* Add trader fee query

* Generated protofile changes

* Add Taker fee

* Add querier logic

* Generated protofile changes

* Add grpc guery.go

* Generated protofile changes

* Update proto/osmosis/poolmanager/v1beta1/query.proto

Co-authored-by: Adam Tucker <[email protected]>

* Update proto/osmosis/poolmanager/v1beta1/query.proto

Co-authored-by: Adam Tucker <[email protected]>

* Add cli

* Generated protofile changes

* register query in query server

---------

Co-authored-by: github-actions <[email protected]>
Co-authored-by: Adam Tucker <[email protected]>
Co-authored-by: Adam Tucker <[email protected]>
@github-actions github-actions bot mentioned this pull request Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:backport/v20.x backport patches to v20.x branch C:CLI C:x/poolmanager V:state/breaking State machine breaking PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants