Skip to content
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

support ibc type for stylus-sdk-rs #867

Open
DaviRain-Su opened this issue Sep 15, 2023 · 3 comments
Open

support ibc type for stylus-sdk-rs #867

DaviRain-Su opened this issue Sep 15, 2023 · 3 comments
Labels
A: low-priority Admin: low priority / non urgent issue, expect longer wait time for PR reviews O: new-feature Objective: aims to add new feature

Comments

@DaviRain-Su
Copy link
Contributor

Feature Summary

relate: OffchainLabs/stylus-sdk-rs#51

Proposal

@Farhad-Shabani
Copy link
Member

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.

@Farhad-Shabani Farhad-Shabani added O: new-feature Objective: aims to add new feature A: low-priority Admin: low priority / non urgent issue, expect longer wait time for PR reviews labels Sep 22, 2023
@DaviRain-Su
Copy link
Contributor Author

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

@Farhad-Shabani
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: low-priority Admin: low priority / non urgent issue, expect longer wait time for PR reviews O: new-feature Objective: aims to add new feature
Projects
Status: 📥 To Do
Development

No branches or pull requests

2 participants