-
Notifications
You must be signed in to change notification settings - Fork 31
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
Implement ICS 721 #167
Implement ICS 721 #167
Conversation
@yito88 would you regenerate protos given that the proto compiler has been revised as of |
b60972f
to
2971bf7
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.
We just need to address the issue with the cargo doc
. It can be patched automatically the same way we do that for the ICS-20:
ibc-proto-rs/tools/proto-compiler/src/cmd/compile.rs
Lines 225 to 231 in aecfe0d
( | |
"ibc.applications.transfer.v1.rs", | |
&[( | |
"The denomination trace (\\[port_id\\]/[channel_id])+/\\[denom\\]", | |
"The denomination trace `([port_id]/[channel_id])+/[denom]`", | |
)], | |
), |
Once that's resolved, everything else appears good, and we can proceed with merging this PR.
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 @yito88, for sorting things out! 🎉
Import proto from https://github.com/bianjieai/nft-transfer for ICS-721