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

Add ibc3 feature #37

Closed
wants to merge 1 commit into from
Closed

Conversation

Art3miX
Copy link
Collaborator

@Art3miX Art3miX commented Nov 20, 2022

From what i know, ibc3 is the norm in cosmos, including this feature seems to be the logic thing to do in ics721.

Small fixes to unit-tests, ibc_channel_open change of expected result, and upgrade optimizer version to 0.12.9

Closes #36

@jhernandezb
Copy link
Member

pretty much every chain is now running ibc v3 and v4 coming with the new wasmd upgrade so I guess this is fine.

Copy link
Contributor

@0xekez 0xekez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏 lg2m. thank you for noticing and fixing this!

@@ -31,10 +31,13 @@ pub enum ExecuteMsg {
}

#[cw_serde]
#[derive(QueryResponses)]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you :)

@0xekez
Copy link
Contributor

0xekez commented Dec 28, 2022

i've moved this commit into my latest changes: 1b086c6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ibc3 feature failing tests
3 participants