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

List and doc all supported capabilities #1442

Closed
alpe opened this issue Sep 28, 2022 · 5 comments · Fixed by #1503
Closed

List and doc all supported capabilities #1442

alpe opened this issue Sep 28, 2022 · 5 comments · Fixed by #1503

Comments

@alpe
Copy link
Contributor

alpe commented Sep 28, 2022

Follow up from CosmWasm/wasmd#980 (comment)

It would be nice to have a .md file that contains a list and description of available capabilities. I have a hard time myself to find them

@uint
Copy link
Contributor

uint commented Nov 22, 2022

Would it be sufficient to add missing "built-in" capabilities to this list?

- `iterator` is for storage backends that allow range queries. Not all types of
databases do that. There are trees that don't allow it and Secret Network does
not support iterators for other technical reasons.
- `stargate` is for messages and queries that came with the Cosmos SDK upgrade
"Stargate". It primarily includes protobuf messages and IBC support.
- `staking` is for chains with the Cosmos SDK staking module. There are Cosmos
chains that don't use this (e.g. Tgrade).

@webmaster128
Copy link
Member

webmaster128 commented Nov 22, 2022

Yes. However, I'd go a small step further and call them "well known capabilities" (or "built-in capabilities", not sure what is best) and extract the list into a new document. This way we have dedicated links that describe the concept and list the capabilities designed by us.

@webmaster128
Copy link
Member

I just made a small edit on naming. Not sure what's best. But "Common" is too generic.

@uint
Copy link
Contributor

uint commented Nov 22, 2022

Yeah, I was thinking of changing it to "all built-in capabilities" too. It seems more accurate.

@webmaster128
Copy link
Member

+1 for "built-in capabilities". I like it more than "well known".

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