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 missing RPC function and make proto file readable #1081

Merged
merged 5 commits into from
Aug 23, 2022

Conversation

liorbond
Copy link
Contributor

No description provided.

Copy link
Member

@Cashmaney Cashmaney left a comment

Choose a reason for hiding this comment

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

When using addresses as "github.com/cosmos/cosmos-sdk/types.AccAddress" they are encoded in url unsafe base64 and cannot be used safely in the query - some will fail, and the others will be uncoded in a weird way. Use "string" instead to get the bech32 encoded address that we're looking for

@liorbond liorbond requested a review from Cashmaney August 23, 2022 19:37
@liorbond liorbond merged commit 1b9735e into lior-expose-all-in-rpc Aug 23, 2022
@liorbond liorbond deleted the lior-add-missing-rpc branch August 23, 2022 20:02
@assafmo
Copy link
Member

assafmo commented Sep 6, 2022

Replaced by #1130

@assafmo assafmo mentioned this pull request Sep 6, 2022
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.

3 participants