Documentation of /bank/balances does not match implementation

[webmaster128]

Summary of Bug

The result of a call to the LCD endpoint /bank/balances looks like

$ curl -sS http://localhost:1317/bank/balances/cosmos1cjsxept9rkggzxztslae9ndgpdyt2408lk850u | jq
{
  "height": "255",
  "result": [
    {
      "denom": "ucosm",
      "amount": "1000000000"
    },
    {
      "denom": "ustake",
      "amount": "1000000000"
    }
  ]
}

However, the output is documented like this:

[
  {
    "denom": "stake",
    "amount": "50"
  }
]

The "ucosm" and "ustake" denoms are from wasmd. The problem is the structure, not the values.

Version

0.38

Steps to Reproduce

---

For Admin Use

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

[alexanderbez]

@webmaster128 we've longggg ago stopped supporting any manual updates or maintenance to the swagger doc file, which was a huge PIA. We've attempted a stab at an automated solution here but we came across many roadblocks.

This happened before we started the Protobuf migration. So when we started Protobuf migration, we decided to ditch those efforts and use the Swagger documentation capabilities that'll come with JSON HTTP gateway.

So by all means you can keep this issue open, but we're not going to manually update the swagger doc.

[webmaster128]

I see. Yeah, today I already found more issues than I can document. In this case I think a big warning like

This document is outdated beyond all hope and only serves as a rough sketch of what to expect from the LCD API. With the Stargate upgrade, we'll utilize the automated Swagger documentation capabilities that come with grpc-gateway (a JSON over HTTP proxy around grpc).

on https://cosmos.network/rpc

[tac0turtle]

swagger doc is now autogenerated and proto based.