-
Notifications
You must be signed in to change notification settings - Fork 3
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
Collin/queries #22
Collin/queries #22
Conversation
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.
Few points for discussion. Thanks for putting this together!
} | ||
|
||
Ok(uri.to_string()) | ||
} |
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.
I think this method is sketchy and would appreciate some feedback on it @EricBolten @tony-iqlusion
Establishes cosmos module exposure with convenient rust module wrapping
Exposes module query clients with convenience type wrappers
Implements convenience methods for basic bank queries
Unit/integration tests
Closes #5