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

Improvement ideas for Core #148

Open
7 tasks
webmaster128 opened this issue Aug 28, 2024 · 0 comments
Open
7 tasks

Improvement ideas for Core #148

webmaster128 opened this issue Aug 28, 2024 · 0 comments

Comments

@webmaster128
Copy link
Member

After reading through the entire Core part (which is great by the way!), I noted a few things that could be improved

Installation

  • Get rid of cargo-run-script

Entrypoints

  • Explain required and optional entry points (I think only instantiate is required at this point, not sure)
  • Show the typical message format for instantiate, migrate and execute. Highlight the use of an enum in execute

Actor Model

  • Worth mentioning that this plays nicely with Cosmos SDK messages. E.g. you don’t have access to the bank keeper to directly manipulate messages but emit a MsgSend instead.

Events

  • The structure of an event missing, i.e. a type fiels plus a list of attributes. The event itself has no keys, which is unclear from the text.

Pinning

  • Explain how to get hit stats for pinned contracts in CosmWasm 2.1 and above

Standard Library/Math

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

No branches or pull requests

1 participant