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.

Sign up for GitHub

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

Jump to bottom

Key terminology changes. Also remove some unused macros. fixes #44 #46

Merged
merged 2 commits into from
Apr 19, 2016
Merged

Key terminology changes. Also remove some unused macros. fixes #44 #46

merged 2 commits into from
Apr 19, 2016

Conversation

daira
Copy link
Collaborator

@daira daira commented Apr 18, 2016

Signed-off-by: Daira Hopwood daira@jacaranda.org

@daira
Key terminology changes. Also remove some unused macros. fixes #44
2a322f8 
Signed-off-by: Daira Hopwood <daira@jacaranda.org>
@daira daira mentioned this pull request Apr 18, 2016
Closed
@daira
Try rerendering the key_components diagram with "Convert text to path…
465ce90 
…s", to

work around a github PDF renderer issue.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>
@daira daira merged commit 802534e into master Apr 19, 2016
@daira daira deleted the zips44.keys-are-not-public.0 branch April 21, 2018 13:20
daira pushed a commit to daira/zips that referenced this pull request Nov 8, 2023
@vivek-arte
Clarifying the encoding of ik (zcash#46)
1b29061 
This makes it clear that the encoding of `ik` used in the Asset Base
derivation is big-endian, as in the case of the underlying BIP 340
Schnorr signature scheme.

It also adds a clarification about the version byte used in the Asset
Base derivation.
daira pushed a commit to daira/zips that referenced this pull request Feb 7, 2024
@vivek-arte @daira
Clarifying the encoding of ik (zcash#46)
9eb00ee 
This makes it clear that the encoding of `ik` used in the Asset Base
derivation is big-endian, as in the case of the underlying BIP 340
Schnorr signature scheme.

It also adds a clarification about the version byte used in the Asset
Base derivation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@daira