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

SubMsgResponse::msg_responses empty with "cosmwasm_2_0" #206

Open
Buckram123 opened this issue Sep 5, 2024 · 3 comments · May be fixed by #213
Open

SubMsgResponse::msg_responses empty with "cosmwasm_2_0" #206

Buckram123 opened this issue Sep 5, 2024 · 3 comments · May be fixed by #213
Assignees
Milestone

Comments

@Buckram123
Copy link

Buckram123 commented Sep 5, 2024

cw-multi-test does not fill SubMsgResponse::msg_responses with expected data even when "cosmwasm_2_0" feature is enabled.

@DariuszDepta DariuszDepta self-assigned this Sep 5, 2024
@CyberHoward
Copy link

To be more specific here, the new msg_responses field is not being propagated correctly.

@DariuszDepta
Copy link
Member

Hi @CyberHoward, @Buckram123!
Would it be possible to take a look at your test cases where this issue occurs?
Thanks in advance!

@DariuszDepta DariuszDepta added this to the 2.2.0 milestone Sep 16, 2024
@Buckram123 Buckram123 changed the title Uses of deprecated items in cosmwasm 2.0 SubMsgResponse::msg_responses empty with "cosmwasm_2_0" Sep 16, 2024
@Buckram123
Copy link
Author

Hi @CyberHoward, @Buckram123! Would it be possible to take a look at your test cases where this issue occurs? Thanks in advance!

Hey @DariuszDepta. Thank you for taking care of this issue!

So the issue we are facing right now is SubMsgResponse::msg_responses being empty even with cosmwasm_2_0 enabled. Created small test in #211 that I would expect to pass.

Also updated the title, because this package still need to support older versions of the cosmwasm, and there is no way to avoid use of the deprecated items.

@DariuszDepta DariuszDepta linked a pull request Sep 24, 2024 that will close this issue
@DariuszDepta DariuszDepta linked a pull request Sep 24, 2024 that will close this issue
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 a pull request may close this issue.

3 participants