Remove all "TypeMsg.." constants

[colin-axner]

Summary

Throughout the codebase we have "TypeMsg..." constants such as:

• TypeMsgTransfer
• TypeMsgCreateClient
• etc

These should be removed

Problem Definition

These constants are a relic of previous SDK functionality which used a string returned from a "Type()" function to route messages internally. The SDK now uses gRPC and thus the proto url to route messages. These constants should be unused and have no intended meaning at the moment

Proposal

Remove all "TypeMsg..." constants. If they are being used somewhere, replace with appropriate usage

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged/assigned

[crodriguezvega]

@colin-axner is it ok then that we remove the implementations of the Type() functions that are remaining (like here for transfer?

[alizahidraja]

Lmk if I can work on this 👋

[crodriguezvega]

Yes, thank you, @alizahidraja!