regen-network / cosmos-modules

Other
10 stars 2 forks source link

Replace amino with protobuf #4

Closed alpe closed 4 years ago

alpe commented 4 years ago

Replaces amino with protobuf

Note to reviewer: GroupMetadata and GroupMember are now defined in protobuf. This makes them first class objects in the orm package. A better solution would be to move them and all the depending tests to orm_test package. I would do this as a second step when 👍

aaronc commented 4 years ago

GroupMetadata and GroupMember are now defined in protobuf. This makes them first class objects in the orm package. A better solution would be to move them and all the depending tests to orm_test package. I would do this as a second step when 👍

I think we could also just start doing this in the actual group package, unless doing so would mean there isn't actually enough test coverage for orm.