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

Escrow Concepts #111

Merged
merged 15 commits into from
Jun 5, 2019
Merged

Escrow Concepts #111

merged 15 commits into from
Jun 5, 2019

Conversation

TheTaconator
Copy link
Contributor

@TheTaconator TheTaconator commented Oct 12, 2018

Latest version of formatted document can be read here

Original Issue

Original pull request moved to new branch due to various file conflicts

…ent) might be required depending on the concept, and the status of the concept
Introduction of new Concept B that improves on Concept A yet is weaker than Concept C
Escrow: Steem Escrow

Escrow: Peerplays Escrow

Escrow: Concept D for compatibility with InterLedger Protocol
Comparison with HTLC

Summary section

References to related discussions, documents, and code
@TheTaconator TheTaconator changed the title Updated draft of escrow concepts BSIP Escrow Concepts Oct 12, 2018
@xeroc
Copy link
Member

xeroc commented Oct 16, 2018

Very nice summary!


This document is placed in the public domain.

# See Also
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestions:

  • strike InterLedger reference
  • strike BSIP40 reference (duplicate of first entry)

README.md Outdated
[45](bsip-0045.md) | Introduce 'allow use as bitasset backing collateral' flag/permission to assets | Customminer | Protocol | Draft
Copy link
Contributor

Choose a reason for hiding this comment

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

Please strike the README.md file from this PR. I will update that file separately.

@pmconrad
Copy link
Contributor

I believe this is almost complete. @TheTaconator please address Ryan's latest comments so we can merge it.

@ryanRfox ryanRfox merged commit f27cede into master Jun 5, 2019
@abitmore abitmore deleted the bsip46 branch August 8, 2019 19:43
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.

4 participants