Remove RegisterRESTRoutes from AppModuleBasic

[ValarDragon]

Summary

Can we remove RegisterRESTRoutes from the AppModuleBasic interface?

RegisterRESTRoutes as far as I know has been unused since the Stargate upgrade, and has been left in the SDK for compatibility reasons.

Problem Definition

Trim the AppModuleBasic / AppModule interface boilerplate, each of which contributes to boilerplate in every SDK module.

Proposal

Delete RegisterRESTRoutes from the AppModuleBasic interface.

---

For Admin Use

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

[tac0turtle]

This makes sense to me, it was/is there to allow apps to still use rest but seems everyone migrated to grpc-gateway already

[alexanderbez]

@facundomedica wanna tackle this quick PR when you get a chance?

[facundomedica]

Sure 😄 @alexanderbez from the interface + all definitions in modules?

[alexanderbez]

Yup!