Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

ConnectionMsg::ConnectionOpen{Try, Ack} should not wrap a Box #258

Closed
plafer opened this issue Nov 22, 2022 · 0 comments · Fixed by #285
Closed

ConnectionMsg::ConnectionOpen{Try, Ack} should not wrap a Box #258

plafer opened this issue Nov 22, 2022 · 0 comments · Fixed by #285
Labels
A: good-first-issue Admin: good for newcomers O: new-feature Objective: aims to add new feature

Comments

@plafer
Copy link
Contributor

plafer commented Nov 22, 2022

Remove Box here.
@plafer plafer added O: new-feature Objective: aims to add new feature A: good-first-issue Admin: good for newcomers labels Nov 22, 2022
@DaviRain-Su DaviRain-Su mentioned this issue Dec 6, 2022
Merged
7 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
A: good-first-issue Admin: good for newcomers O: new-feature Objective: aims to add new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove ConnectionMsg variant connection{Try, Ack} Box Wrap octopus-network/ibc-rs
1 participant
@plafer