Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

Add cw1 section and subkeys doc #87

Merged
merged 4 commits into from
Aug 25, 2020
Merged

Add cw1 section and subkeys doc #87

merged 4 commits into from
Aug 25, 2020

Conversation

orkunkl
Copy link
Contributor

@orkunkl orkunkl commented Aug 24, 2020

Related to CosmWasm/testnets#51
Resolves #83

I kept it simple and language is funny. I can make it formal if requested.

@orkunkl orkunkl requested a review from ethanfrey August 24, 2020 16:16
Copy link
Member

@ethanfrey ethanfrey left a 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
Copy link
Member

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)

Copy link
Contributor Author

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.


```ts
// contract is already uploaded on coralnet: codeId -> 12
// const codeId = await factory.upload();
Copy link
Member

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)

// contract is already uploaded on coralnet: codeId -> 12
// const codeId = await factory.upload();
const { address } = await client.getAccount()
// contract.contractAddress -> 'coral1267wq2zk22kt5juypdczw3k4wxhc4z47mug9fd'
Copy link
Member

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:
Copy link
Member

Choose a reason for hiding this comment

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

😆

@@ -1,6 +1,6 @@
---
title: Advanced Usage
order: 3
order: 4
Copy link
Member

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice idea.

@orkunkl orkunkl merged commit 8c6e1e3 into master Aug 25, 2020
@orkunkl orkunkl deleted the cw1-docs branch August 25, 2020 12:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document cw1-subkeys usage
2 participants