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

da-schnorr signatures #125

Merged
merged 4 commits into from Feb 21, 2019
Merged

da-schnorr signatures #125

merged 4 commits into from Feb 21, 2019

Conversation

anselld
Copy link
Contributor

@anselld anselld commented Feb 15, 2019

Hi Cayle. Please could you review my suggested formatting changes to your three sections. They now fall under one heading: "Introduction to Schnorr Signatures".
Please let me know if you would like any changes before I start the editing.
Please could you help with the date [9] was accessed. I still need to insert all the authors for [9] - I'll include them when I do the editing.
Thanks.

@hansieodendaal hansieodendaal removed their request for review February 15, 2019 11:58
@anselld
Copy link
Contributor Author

anselld commented Feb 20, 2019

@CjS77
Hi Cayle, I've inserted the authors for [9]. Please let me know if you're happy with the formatting before I start with the editing. Thanks.

Copy link
Member

@CjS77 CjS77 left a comment

Choose a reason for hiding this comment

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

LGTM. I think some of the TOC headings might change as a result of the merging; but let's cross that bridge when we get there.

@CjS77 CjS77 merged commit afb20c5 into master Feb 21, 2019
@CjS77 CjS77 deleted the da-schnorr-signatures branch February 21, 2019 11:41
@hansieodendaal hansieodendaal restored the da-schnorr-signatures branch February 21, 2019 11:56
@hansieodendaal hansieodendaal deleted the da-schnorr-signatures branch February 21, 2019 11:57
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.

None yet

3 participants