-
Notifications
You must be signed in to change notification settings - Fork 3.8k
feat(docs): union ibc spec #3201
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
8d7e891 to
1304a23
Compare
Docs 🤌✨ Deployment complete! Take a peek over at https://8a75bd2f.docs-ajn.pages.dev |
1304a23 to
bee0ff4
Compare
aeryz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We also changed recvPacket to get multiple packets instead of one.
bee0ff4 to
82585ab
Compare
82585ab to
899e1de
Compare
899e1de to
ec32143
Compare
|
|
||
| - [`connectionId`](https://github.com/cosmos/ibc/tree/41a8caa54d8691b7fa98795d31f401e1df31e18c/spec/core/ics-003-connection-semantics#identifier-validation) is no longer a string but a unique `uint32`. | ||
| - [`ConnectionEnd`](https://github.com/cosmos/ibc/tree/41a8caa54d8691b7fa98795d31f401e1df31e18c/spec/core/ics-003-connection-semantics#data-structures) | ||
| - `counterpartyPrefix` has been removed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps explain that this is to be done in the light client now?
| - [`Packet`](https://github.com/cosmos/ibc/blob/41a8caa54d8691b7fa98795d31f401e1df31e18c/spec/core/ics-004-channel-and-packet-semantics/README.md#definitions) | ||
| - `sourcePort` has been removed. | ||
| - `destinationPort` has been removed. | ||
| - `timeoutHeight` type has been changed from `(uint64, uint64)` to `uint64`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Explain that revision number is removed
No description provided.