Carlos Rodriguez

For IBC I would recommend that you remove all content from the SDK docs and you just have a link to redirect the user to [ibc-go's own documentation website](https://ibc.cosmos.network).

Hi @odeke-em, isn't the length specification right below the highlighted text on the screenshot you attached in the issue? 

In [ibc-go we have validation](https://github.com/cosmos/ibc-go/blob/main/modules/core/24-host/validate.go#L62-L88) to make sure the identifiers are in the length range specified in the spec. I think no changes are required in the spec, so the...

In ibc-go we actually [return an error](https://github.com/cosmos/ibc-go/blob/main/modules/core/exported/client.go#L98). For 07-tendermint the implementation looks like [this](https://github.com/cosmos/ibc-go/blob/main/modules/light-clients/07-tendermint/update.go#L21-L33).

Thanks for opening this issue, @plafer. When I opened the PR to add the support for base denoms with slashes, I thought that it was a bug on ibc-go that...

Thanks for pointing this out, @plafer! We discussed with @cwgoes and given that (if I'm not mistaken) [ibc-rs is also using a structure for `Version`](https://github.com/cosmos/ibc-rs/blob/main/crates/ibc/src/core/ics03_connection/version.rs#L13-L20), then it was agreed that...

> Does the spec need to be updated in this case? @AdityaSripal do you think a check for the timeout timestamp should be added in the spec? In the implementation...

Thanks for the explanation, @AdityaSripal! Then I believe we can close this issue, since there are no changes needed in ibc-go, right? And as Aditya suggested, you're welcome to open...

Hi @hu55a1n1. Just wondering if you had any time to work on this change. Feel free to delegate to us if you're too busy with other things!

> there should probably be a check enforcing that controller chain ports cannot receive packets @colin-axner, is then the action for this issue to update the spec to include the...