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

feat: support schnorr_1 for zilliqa #257

Merged
merged 1 commit into from
Dec 1, 2020

Conversation

renlulu
Copy link
Contributor

@renlulu renlulu commented Dec 1, 2020

Motivation

This PR is to support Zilliqa schnorr signature algorithm.

Solution

Add Sign and Verify functions for secp256k1 by using zilliqa golang sdk.

patrick-ogrady
patrick-ogrady previously approved these changes Dec 1, 2020
Copy link
Contributor

@patrick-ogrady patrick-ogrady left a comment

Choose a reason for hiding this comment

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

🚀

@patrick-ogrady
Copy link
Contributor

@renlulu it won't let me merge this until you sign your commits 😢
image

@renlulu
Copy link
Contributor Author

renlulu commented Dec 1, 2020

@patrick-ogrady Oh ok! I will redo this PR.

@patrick-ogrady
Copy link
Contributor

Looks good otherwise!

@renlulu
Copy link
Contributor Author

renlulu commented Dec 1, 2020

@patrick-ogrady I signed it. Hope you don't mind my force push here, or I can do a fresh PR.

@patrick-ogrady patrick-ogrady merged commit 0ea61c9 into coinbase:master Dec 1, 2020
@patrick-ogrady
Copy link
Contributor

force push is fine!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants