Better ModuleAccount JSON serialization

[RiccardoM]

Summary

Improve the way ModuleAccount is serialized to JSON

Problem Definition

Currently a ModuleAccount is serialized as follows:

{
  "BaseAccount": {
    "address": "did:com:154s52fcvu6ema0ga6qftww2gva7avxx54hy2p5",
    "coins": [
      {
        "denom": "ucommercio",
        "amount": "20"
      }
    ],
    "public_key": null,
    "account_number": "7",
    "sequence": "0"
  },
  "name": "id",
  "permissions": null
}

The BaseAccount is due to Amino not following json encoding standards about wrapped types.
I think this can be easily and safely dropped and doing so will also allow easier reading.

Proposal

I think it should be better to implement a custom serializer that allows to write the a ModuleAccount instance as

{
  "address": "did:com:154s52fcvu6ema0ga6qftww2gva7avxx54hy2p5",
  "coins": [
    {
      "denom": "ucommercio",
      "amount": "20"
    }
  ],
  "public_key": null,
  "account_number": "7",
  "sequence": "0",
  "name": "id",
  "permissions": null
}

---

For Admin Use

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

[alexanderbez]

By all means, please implement a JSON marshall/unmarshal for the ModuleAccount type. Maybe even the standard account types too.