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

fix: block results validator update decoder #1151

Merged

Conversation

blorgon1
Copy link
Contributor

Closes #1142.

@blorgon1 blorgon1 force-pushed the fix/block-results-validator-updates branch from 90d8c9f to 784fc0c Compare May 23, 2022 15:22
@blorgon1 blorgon1 force-pushed the fix/block-results-validator-updates branch from 87cfb14 to 8296ca0 Compare May 24, 2022 16:12
@blorgon1
Copy link
Contributor Author

Hi @webmaster128, are there any more changes you would like on this PR before merging? Please let me know, thanks!

@webmaster128
Copy link
Member

In #1162 I added tests for the results of the decoding. I could not push here. The rest is 👍 thank you! Will merge when CI is happy.

@webmaster128 webmaster128 merged commit 8296ca0 into cosmos:main Jun 8, 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.

Uncaught Error: unknown pubkey type: undefined
2 participants