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

HIP 29: M-of-N Multisig #154

Merged
merged 9 commits into from
Apr 22, 2021
Merged

HIP 29: M-of-N Multisig #154

merged 9 commits into from
Apr 22, 2021

Conversation

xandkar
Copy link
Contributor

@xandkar xandkar commented Apr 19, 2021

abhay and others added 2 commits April 19, 2021 12:05
Co-authored-by: Andrew Thompson <andrew@hijacked.us>
0029-multisignature-keys.md Outdated Show resolved Hide resolved
0029-multisignature-keys.md Outdated Show resolved Hide resolved
0029-multisignature-keys.md Outdated Show resolved Hide resolved
@xandkar xandkar marked this pull request as ready for review April 20, 2021 13:51
@xandkar xandkar changed the title Multi-signature addresses HIP 29: M-of-N Multisig Apr 20, 2021
@abhay
Copy link
Contributor

abhay commented Apr 21, 2021

RFC

@jamiew
Copy link
Contributor

jamiew commented Apr 22, 2021

Draft looks good to me. Will merge for discussion. This will be HIP 29

@jamiew jamiew merged commit 1bca118 into master Apr 22, 2021
@jamiew
Copy link
Contributor

jamiew commented Apr 22, 2021

This HIP draft has been accepted and merged for discussion. Please direct comments & feedback to the tracking issue: #157

@jamiew jamiew deleted the sk/multisig branch April 22, 2021 17:53
@Jamill-hallak
Copy link

I think when you want to make M-of-N multisig , we should build some functions to get a special private key from other basic private keys .so is the code of Helium have some functions to build a special private key from other basic private keys?
if yes: can you show me the resource code of it?
if not: I can build some functions like this, but here I need to know all about the standard building of helium's private key.

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.

5 participants