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

BIP 197: Hashed Time-Locked Collateral Contract #773

Merged
merged 3 commits into from
Mar 29, 2019

Conversation

matthewjablack
Copy link
Contributor

@matthewjablack matthewjablack commented Mar 27, 2019

Hello all,

This proposal went through the mailing list: https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2019-March/016805.html without any further discussion. Currently this proposal has been implemented by in the ChainAbstractionLayer (https://github.com/liquality/chainabstractionlayer/tree/dev). I would like to kindly ask for BIP # assignment and have this merged into upstream bips.

Cheers,

Matthew Black

@matthewjablack matthewjablack changed the title BIP 200: Hashed Time-Locked Collateral Contract transactions BIP 200: Hashed Time-Locked Collateral Contract Mar 27, 2019
@matthewjablack matthewjablack changed the title BIP 200: Hashed Time-Locked Collateral Contract BIP ???: Hashed Time-Locked Collateral Contract Mar 27, 2019
@matthewjablack matthewjablack changed the title BIP ???: Hashed Time-Locked Collateral Contract BIP 200: Hashed Time-Locked Collateral Contract Mar 27, 2019
@luke-jr luke-jr changed the title BIP 200: Hashed Time-Locked Collateral Contract New BIP: Hashed Time-Locked Collateral Contract Mar 28, 2019
@luke-jr
Copy link
Member

luke-jr commented Mar 28, 2019

Never self-assign BIP numbers.

@luke-jr
Copy link
Member

luke-jr commented Mar 28, 2019

(BIP 200 is already reserved for something else)

bip-0200.mediawiki Outdated Show resolved Hide resolved
Comments-Summary: No comments yet.
Comments-URI: TBD
Status: Proposal
Type: Informational
Copy link
Member

Choose a reason for hiding this comment

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

This should be Standards Track

Atomic Loans team <[email protected]>
Comments-Summary: No comments yet.
Comments-URI: TBD
Status: Proposal
Copy link
Member

Choose a reason for hiding this comment

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

This should be Draft

@luke-jr
Copy link
Member

luke-jr commented Mar 28, 2019

For a BIP number to be assigned, you must add a Backwards compatibility section.

bip-0200.mediawiki Outdated Show resolved Hide resolved
@luke-jr luke-jr changed the title New BIP: Hashed Time-Locked Collateral Contract BIP 197: Hashed Time-Locked Collateral Contract Mar 29, 2019
@luke-jr
Copy link
Member

luke-jr commented Mar 29, 2019

Please use BIP number 197 (and purge the incorrect number from the git branch history)

@matthewjablack matthewjablack force-pushed the htlcc branch 8 times, most recently from 5eeb2d1 to 4809519 Compare March 29, 2019 16:46
Title: Hashed Time-Locked Collateral Contract
Author: Matthew Black <[email protected]>
Tony Cai <[email protected]>
Atomic Loans team <[email protected]>
Copy link
Member

Choose a reason for hiding this comment

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

Not sure teams can be authors; either way, this needs to match what's in README

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed Atomic Loans team <[email protected]> from Author list

<pre>
BIP: 197
Layer: Applications
Title: Hashed Time-Locked Collateral Contract
Copy link
Member

Choose a reason for hiding this comment

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

Needs to match README

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed title in README for BIP 197 to Hashed Time-Locked Collateral Contract

@luke-jr luke-jr merged commit 49e38e2 into bitcoin:master Mar 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants