search

cosmos / ibc-rs

Rust implementation of the Inter-Blockchain Communication (IBC) protocol.
Apache License 2.0
197 stars 79 forks source link

support ibc type for stylus-sdk-rs #867

Open DaviRain-Su opened 1 year ago

DaviRain-Su commented 1 year ago

Feature Summary

relate: https://github.com/OffchainLabs/stylus-sdk-rs/issues/51

Proposal

Farhad-Shabani commented 1 year ago

Thanks @DaviRain-Su for bringing up this issue and shedding light on other potential uses for ibc-rs.

One thing that immediately caught my attention is the increasing number of derived attributes on IBC types! Managing them this way can become quite challenging, leading to the risk of either unnecessary or overlooked derives for certain types. it sounds necessary we address this as a preliminary step.

By the way, I took a quick look at how we can support, and it seems that safely using the Stylus SDK macro would require us to allocate a portion of our bandwidth to it. We currently lack the necessary bandwidth, both for development and review. I mark this issue as low-priority for the time being.

DaviRain-Su commented 1 year ago

I can support this feature, but need to clear way, how to implement this feature?

Farhad-Shabani commented 1 year ago

I can support this feature, but need to clear way, how to implement this feature?

Really appreciate your offer. Honestly, at this time have no clear idea around it. It demands a prelim study. Let's see how things go the coming quarter, and I'll let you know if this can be done.