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

[Bug]: Bank Module DenomOwners returns not implemented on osmosis #7353

Closed
hindsight9923 opened this issue Jan 21, 2024 · 1 comment
Closed

Comments

@hindsight9923
Copy link

hindsight9923 commented Jan 21, 2024

What happened?

When testing #6847 using both gRPC and lcd with an ibc denomination and uosmo both attempts return a "not implemented on osmosis. use an indexer" status. Removing the implementation of this bank module function was not listed in scope of the most recent upgrade.

image
url - https://lcd.osmosis.zone/cosmos/bank/v1beta1/denom_owners/uosmo

Osmosis Version

main

How to reproduce?

url - https://lcd.osmosis.zone/cosmos/bank/v1beta1/denom_owners/uosmo
Execute gRPC call using denomOwners method on osmosis returns the same "not implemented" results

@ValarDragon
Copy link
Member

ValarDragon commented Feb 17, 2024

It is not implemented in osmosis from the state machine and plan to never include jt. The design decision to include is a significant flaw imo as it slows down important core logic.

Getting this query should come from a new indexer stack, not the base full nodes themselves

@github-project-automation github-project-automation bot moved this from Needs Triage 🔍 to Done ✅ in Osmosis Chain Development Feb 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

2 participants