-
Notifications
You must be signed in to change notification settings - Fork 40
feat: derive Keystore public key from secret key #1067
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments. Let me know if you're having problems!
hey @MegaRedHand sorry for the late pr. But i have fixed the issue , let me know if you need any changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left you some more comments. Good work so far!
Thank you and add the necessary changes. Not sure why the 2nd commit happened. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left you some more comments.
added the fix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! Please mark the PR as "ready for review" so we can merge it.
Thank you @MegaRedHand , for helping me out. |
Motivation
Description
addresses #1028
Having an error in deserialization. deserialization accepts u8 but private key is of type Binary. In other functions it works fine.
Wondering where I went wrong.