Skip to content

New topic: vaults#411

Merged
jnewbery merged 1 commit into
bitcoinops:masterfrom
harding:2020-06-new-topic-vaults
Jun 16, 2020
Merged

New topic: vaults#411
jnewbery merged 1 commit into
bitcoinops:masterfrom
harding:2020-06-new-topic-vaults

Conversation

@harding

@harding harding commented Jun 1, 2020

Copy link
Copy Markdown
Collaborator

No description provided.

@bitschmidty bitschmidty left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @harding couple small suggestions

Comment thread _topics/en/vaults.md Outdated
Comment thread _topics/en/vaults.md Outdated
@jnewbery jnewbery force-pushed the 2020-06-new-topic-vaults branch from 3f0ecc8 to 9f4c5f6 Compare June 4, 2020 14:49
@jnewbery

jnewbery commented Jun 4, 2020

Copy link
Copy Markdown
Contributor

I've rebased and added the vaults links to the OP_CHECKSIGFROMSTACK and OP_CHECKTEMPLATEVERIFY topics pages.

@harding harding mentioned this pull request Jun 15, 2020
3 tasks
@harding harding force-pushed the 2020-06-new-topic-vaults branch from 9f4c5f6 to 1d10c3c Compare June 15, 2020 14:41
@bitschmidty

Copy link
Copy Markdown
Contributor

ACK 1d10c3c

@jnewbery jnewbery left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. A few very minor suggested changes.

Comment thread _topics/en/vaults.md Outdated
## Required. Use Markdown formatting. Only one paragraph. No links allowed.
## Should be less than 500 characters
excerpt: >
**Vaults** are a type of covenant that requires two separate

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should this be 'require' to agree with 'vaults'. Alternatively: "A vault is a type of covenant that requires"

Comment thread _topics/en/vaults.md Outdated

## Optional. Use Markdown formatting. Multiple paragraphs. Links allowed.
extended_summary: |
Usually, there's a minimum amount of time or number of blocks that

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There seems to be some difference between the excerpt "transactions ... appear in two different blocks" and this "Usually, there's a minimum amount of time or blocks".

The first states that the transactions must appear in different blocks. The second implies with the "usually" that there could be cases where the transactions appear in the same block.

Comment thread _topics/en/vaults.md Outdated
Some vault designs rely on [covenants][topic covenants] that require
consensus changes to Bitcoin. Other vault designs use existing
protocol features plus techniques such as multisig, presigned
transactions, and ephemeral private keys.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think 'ephemeral' emphasizes the right thing. The critical part is that the keys are deleted.

@harding harding force-pushed the 2020-06-new-topic-vaults branch from 1d10c3c to cc2e3a7 Compare June 16, 2020 16:03
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.

3 participants