Closed mattverse closed 7 months ago
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
@mattverse can this be revived against the osmo/v0.47.5 branch?
Also, wouldn't it make more sense to just add this directly to osmosis? Since we aren't going to be registering any custom messages generally direct to sdk, and keeps us further from the fork (unless you are going to upstream)
Based on ledger updates per matt, seems like we can actually close!
Description
Closes: https://github.com/osmosis-labs/osmosis/issues/1106, backport https://github.com/cosmos/cosmos-sdk/pull/11349
What is the purpose of the change
Fixes so that RegisterLegacyAminoCodec errors if Msg name is >39 chars, this has been a long bug that has not been addressed for a long time.
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.
I have...
!
to the type prefix if API or client breaking changeCHANGELOG.md
make lint
andmake test
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.
I have...
!
in the type prefix if API or client breaking change