-
Notifications
You must be signed in to change notification settings - Fork 100
Add cw1 section and subkeys doc #87
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice stuff and I like the style.
A few points to address before merging. Thanks for setting this up
skip this section and go to the [next page](./setup.md) | ||
::: | ||
|
||
## Base |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is cut-and-paste from the README, right?
That is fine (and nit to publicize this more), but we need to note to update this if I update the spec (add iterator to list all accounts for example)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then we make sure documentation pipeline is attentive.
cw-plus/cw1/subkeys.md
Outdated
|
||
```ts | ||
// contract is already uploaded on coralnet: codeId -> 12 | ||
// const codeId = await factory.upload(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe remove some of the comments (the one explaining codeId -> 12 is good, the others confusing)
cw-plus/cw1/subkeys.md
Outdated
// contract is already uploaded on coralnet: codeId -> 12 | ||
// const codeId = await factory.upload(); | ||
const { address } = await client.getAccount() | ||
// contract.contractAddress -> 'coral1267wq2zk22kt5juypdczw3k4wxhc4z47mug9fd' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although printing this out and a comment on how to reconnect later (not making a new one) is a good advice
contract.freeze() | ||
``` | ||
|
||
Let's give some allowance to your friends account, so he can buy a lambo: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😆
cw-plus/cw20/advanced.md
Outdated
@@ -1,6 +1,6 @@ | |||
--- | |||
title: Advanced Usage | |||
order: 3 | |||
order: 4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe this belongs in "Overview" or "General" as these tips are valid for all contracts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice idea.
Related to CosmWasm/testnets#51
Resolves #83
I kept it simple and language is funny. I can make it formal if requested.