Reconsider Code Usage in x/auth Client

[alexanderbez]

Summary

The following was introduced in #5533 to allow x/auth to be used when needing to instantiate and use AccountRetriever. This would require an app to manually set this variable.

// x/auth/client/tx.go

// Codec defines the x/auth account codec to be used for use with the
// AccountRetriever. The application must be sure to set this to their respective
// codec that implements the Codec interface and must be the same codec that
// passed to the x/auth module.
//
// TODO:/XXX: Using a package-level global isn't ideal and we should consider
// refactoring the module manager to allow passing in the correct module codec.
var Codec authtypes.Codec

Instead, we should discover a better solution that avoids needing to manually override a variable. Note, this would be applicable for any modules who's client logic relies on serializing interfaces.

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned

[github-actions]

This issue 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.