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 queries to wasmd section #141

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

Add queries to wasmd section #141

wants to merge 1 commit into from

Conversation

pinosu
Copy link
Contributor

@pinosu pinosu commented Aug 15, 2024

Resolves #140

Copy link

vercel bot commented Aug 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
cosmwasm-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 15, 2024 0:32am

Copy link
Contributor

@uint uint left a comment

Choose a reason for hiding this comment

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

Nice expansion!

For someone coming here without a good understanding of the raw/smart distinction, I think it'd be good to explain it.

I'd mention where appropriate that "raw" means querying the underlying KV storage directly, while "smart" means loading the smart contract and executing one of its "query" endpoints to get the structured data. Or something like that?

If it's there somewhere and I missed it, feel free to call me out on that 😁

@@ -514,9 +514,9 @@ sleep 6
wasmd q tx $(echo "$RESP" | jq -r '.txhash') -o json | jq
```

### Query contract state
### Query Contract State All
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### Query Contract State All
### Query Contract State: All

Just feels a little easier to read when separated like that!

@@ -557,6 +558,62 @@ The output will be similar to the following:
}
```

### Query Contract State Raw
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### Query Contract State Raw
### Query Contract State: Raw

}
```

### Query Contract State Smart
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### Query Contract State Smart
### Query Contract State: Smart


### Query Contract State Smart

To query the state of a WASM contract for specific data, you can use the following command:
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd add a line here that the content of QUERY depends on the specific contract you want to call and add a link to core/entrypoints/query

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.

Add Query raw and smart query in wasmd section
3 participants