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

Validators are now sorted by voting power instead of by address #506

Closed
greg-szabo opened this issue Aug 4, 2020 · 1 comment · Fixed by #565
Closed

Validators are now sorted by voting power instead of by address #506

greg-szabo opened this issue Aug 4, 2020 · 1 comment · Fixed by #565
Assignees
Labels
go Compatibility with Go code
Milestone

Comments

@greg-szabo
Copy link
Member

This is a Tendermint-Go 0.34 compatibility upgrade issue brought by a comment in #305 by Ethan.

@xla xla added the go Compatibility with Go code label Aug 4, 2020
greg-szabo added a commit that referenced this issue Sep 3, 2020
Domain types for protobuf structs (#537)

* Domain types
* DomainType derive macro
* DomainType added to all amino_types

As close as test_validator_set gets without issue #506
@greg-szabo greg-szabo self-assigned this Sep 8, 2020
@greg-szabo
Copy link
Member Author

I'm going to work on this now. I'm basing it on top of #537 which is not merged yet.

brapse pushed a commit that referenced this issue Sep 10, 2020
* Ripped out amino types and replaced them with protobuf types.

* Vote and proposal serialization fix

* fmt fix

* As close as test_validator_set gets without issue #506

* sad fmt noises

* Domain types for protobuf structs (#537)

* Domain types
* DomainType derive macro
* DomainType added to all amino_types

* Minor fixes and cleanup

* Fixed voting and proposals

* Updated CHANGELOG

* Documentation update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Compatibility with Go code
Projects
None yet
2 participants