Skip to content

[ZIP 32] Add registered child key derivation with support for tags#978

Merged
str4d merged 4 commits into
zcash:mainfrom
daira:add-child-key-with-tag
Feb 20, 2025
Merged

[ZIP 32] Add registered child key derivation with support for tags#978
str4d merged 4 commits into
zcash:mainfrom
daira:add-child-key-with-tag

Conversation

@daira
Copy link
Copy Markdown
Collaborator

@daira daira commented Feb 12, 2025

No description provided.

@daira daira requested review from nuttycom and str4d February 12, 2025 18:00
Copy link
Copy Markdown
Contributor

@nuttycom nuttycom left a comment

Choose a reason for hiding this comment

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

ACK 24ac3302c54ee5da6a431222b2519dd5cd5ed56e with minor question

Comment thread zips/zip-0032.rst Outdated
@daira daira force-pushed the add-child-key-with-tag branch 2 times, most recently from ab3b159 to e973ec9 Compare February 12, 2025 19:24
@daira daira marked this pull request as ready for review February 12, 2025 19:37
@daira daira changed the title Add support for tags in arbitrary child key derivation Support tagged ZIP 32 child derivation for registered application protocols Feb 13, 2025
@daira daira changed the title Support tagged ZIP 32 child derivation for registered application protocols Define tagged ZIP 32 child derivation for registered application protocols Feb 13, 2025
@daira daira marked this pull request as draft February 13, 2025 03:16
@daira daira force-pushed the add-child-key-with-tag branch 2 times, most recently from 515f1ed to 167da77 Compare February 13, 2025 15:26
@daira daira changed the title Define tagged ZIP 32 child derivation for registered application protocols Add registered child key derivation with support for tags Feb 13, 2025
@daira daira force-pushed the add-child-key-with-tag branch from 167da77 to ef54363 Compare February 13, 2025 15:36
Copy link
Copy Markdown
Contributor

@nuttycom nuttycom left a comment

Choose a reason for hiding this comment

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

Clarifying the type and meaning of full_width_leaf is blocking.

Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
@daira daira force-pushed the add-child-key-with-tag branch 3 times, most recently from 92a0ffb to 36b09c4 Compare February 13, 2025 19:05
@daira daira marked this pull request as ready for review February 13, 2025 19:12
@daira daira force-pushed the add-child-key-with-tag branch 2 times, most recently from 993a280 to 14ea5d9 Compare February 15, 2025 13:16
@daira daira requested a review from nuttycom February 15, 2025 13:24
@daira daira dismissed nuttycom’s stale review February 15, 2025 13:24

comments addressed

Copy link
Copy Markdown
Contributor

@nuttycom nuttycom left a comment

Choose a reason for hiding this comment

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

ACK

Copy link
Copy Markdown
Collaborator

@str4d str4d left a comment

Choose a reason for hiding this comment

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

Reviewed 14ea5d99e7f05de8b46fa819eaec82847f19cbc0

Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
Comment thread zips/zip-0032.rst Outdated
@daira daira force-pushed the add-child-key-with-tag branch from 133771e to a1cbf65 Compare February 17, 2025 22:52
Copy link
Copy Markdown
Collaborator

@str4d str4d left a comment

Choose a reason for hiding this comment

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

ACK a1cbf65ffc1853fe23b2871727fa6d6cda7fafdb

Copy link
Copy Markdown
Collaborator

@conradoplg conradoplg left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread zips/zip-0032.rst Outdated
@nuttycom nuttycom changed the title Add registered child key derivation with support for tags [ZIP 32] Add registered child key derivation with support for tags Feb 19, 2025
@daira daira force-pushed the add-child-key-with-tag branch 2 times, most recently from 4deb4c4 to ced99d4 Compare February 19, 2025 23:52
daira and others added 3 commits February 19, 2025 23:59
requirement to document the usage in a ZIP.

Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
… the

LaTeX cases environment.

Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
Co-authored-by: Jack Grigg <thestr4d@gmail.com>
@daira daira force-pushed the add-child-key-with-tag branch from ced99d4 to f81f700 Compare February 19, 2025 23:59
…tion.

Co-authored-by: Jack Grigg <jack@electriccoin.co>
Co-authored-by: Kris Nuttycombe <kris@nutty.land>
Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
@daira daira force-pushed the add-child-key-with-tag branch from f81f700 to e601318 Compare February 20, 2025 00:06
@daira
Copy link
Copy Markdown
Collaborator Author

daira commented Feb 20, 2025

This is ready for re-review.

@daira daira requested review from nuttycom and str4d February 20, 2025 00:07
Copy link
Copy Markdown
Collaborator

@str4d str4d left a comment

Choose a reason for hiding this comment

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

ACK e601318, includes the changes suggested by the ZIP Editors (@str4d, @nuttycom, @arya2) during review.

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.

4 participants