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 additional information about submessage data handling to SEMANTICS.md #1639

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

0xekez
Copy link
Contributor

@0xekez 0xekez commented Mar 16, 2023

This is a bit of a tricky case for setting data which burned me recently. Contributing some docs in the hope that future CosmWasm people don't need to figure it out themselves.

@kulikthebird
Copy link
Contributor

kulikthebird commented Dec 6, 2024

It's been a while since you posted this PR, but finally we had time to double-check your change. You are right, in today's version wasmd does not forward the data in scenario like A--exec-->B--exec--->C. In this case A won't receive response from C without B forwarding that data explicitly. There's a bigger docs cleanup and this change will be taken into account in CosmWasm/docs#167. Thank you for pointing that out and for your contribution!

@kulikthebird kulikthebird self-assigned this Dec 6, 2024
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.

2 participants