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
IBC Callbacks #1817
base: main
Are you sure you want to change the base?
IBC Callbacks #1817
Conversation
bd3eab3
to
692b398
Compare
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.
🐎
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.
Thank you for this amazing work.
ack ibcexported.Acknowledgement, | ||
contractAddress string, | ||
) error { | ||
// sender validation makes no sense here, as the receiver is never the sender |
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.
Yes, it might be good to document that probably the contract itself should be doing some validation based on the channel-id and packet data possibly.
It might even make sense to have the contract whitelist certain channel-ids to be able to receive callbacks only on those channels (but this is a nice to have and can make the UX more complicated).
No description provided.