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

Mandatory Multisig #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Mandatory Multisig #30

wants to merge 1 commit into from

Conversation

thelink2012
Copy link
Contributor

Closes #28

Issue: Broadcasting using coinb.in doesn't work properly for these transactions, using blockchain.info though goes just fine. So there's something that should be fixed on the backend I guess.

@OutCast3k
Copy link
Owner

Hmmm. This is really cool.

I will review your code and look into the issue with fixing the backend so it accepts the signed transaction and then accept your request to merge.

Nice work.

@neurocis
Copy link

Agreed, very cool, will be reviewing and testing this tonight for PiggyCoinb.in :)

@weex
Copy link

weex commented Nov 16, 2015

@OutCast3k Coinb.in itself is really awesome and desperately needed if people are going to trade safely in a trustless manner. I was thinking a more general construction with logical and/or in it might be good but thelink2012 nailed it by designing this as multisig with a mandatory component which is the only use-case I need supported and was probably a lot more straightforward to implement. Looking forward to see this merged.

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.

Make it possible to build A+(B or C) multisig scripts
4 participants