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

pos: add a query to find validator by Tendermint address #1344

Merged
merged 2 commits into from
Jun 5, 2023

Conversation

tzemanovic
Copy link
Member

@tzemanovic tzemanovic commented May 10, 2023

adds namadac find-validator --tm-address <>

@tzemanovic tzemanovic force-pushed the tomas/find-validator-by-tm branch 2 times, most recently from ee97d64 to 9b48496 Compare May 10, 2023 08:13
tzemanovic added a commit that referenced this pull request May 11, 2023
@tzemanovic tzemanovic marked this pull request as ready for review May 11, 2023 04:37
@tzemanovic tzemanovic requested a review from Fraccaman May 11, 2023 04:37
@Fraccaman
Copy link
Member

LGTM! thanks :)

Fraccaman
Fraccaman previously approved these changes May 11, 2023
@juped juped self-requested a review May 15, 2023 07:54
Copy link
Member

@juped juped left a comment

Choose a reason for hiding this comment

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

this needs to be SDKized

@cwgoes cwgoes mentioned this pull request May 18, 2023
10 tasks
@tzemanovic
Copy link
Member Author

this needs to be SDKized

updated

Fraccaman added a commit that referenced this pull request May 23, 2023
* origin/tomas/find-validator-by-tm:
  changelog: add #1344
  pos: add a query to find validator by Tendermint address
@Fraccaman Fraccaman mentioned this pull request May 23, 2023
@adrianbrink adrianbrink requested a review from juped May 30, 2023 16:45
@Fraccaman Fraccaman merged commit 72007eb into main Jun 5, 2023
@Fraccaman Fraccaman deleted the tomas/find-validator-by-tm branch June 5, 2023 03:18
@hkey0
Copy link

hkey0 commented Feb 13, 2024

this needs to be SDKized

updated

Function validator_by_tm_addr is not available in rpc. This should be added.

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.

4 participants