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 Client class to serve as a general purpose base without message types #1486

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

webmaster128
Copy link
Member

No description provided.

@jgimeno
Copy link

jgimeno commented Feb 22, 2024

Hi Simon, I wonder if this can also solve the oposite scenario. Imagine that I have some data in proto format coming from a query, in the SDK I can use reflection and the interfaceRegistry to get back at least the json representation of that code.

Do you know if this is something that can be done with cosmjs?

@webmaster128
Copy link
Member Author

Do you know if this is something that can be done with cosmjs?

We designed CosmJS with a lot of type safety in mind. A fully dynamic approach would give you no guarantees at compile time. I think both approaches are valid but a client-library focussed on reflections would be a different projects with different trade-offs. I'm happy if someone builds that but it's out of scope for CosmJS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants