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

Reference draft for upstream PR#987 #101

Draft
wants to merge 20 commits into
base: main
Choose a base branch
from
Draft

Conversation

vivek-arte
Copy link

@vivek-arte vivek-arte commented Mar 11, 2025

This is a reference of PR987 upstream to allow us to view the rendered version.

zcash#987

str4d and others added 20 commits February 26, 2025 00:46
It can always be derived, and would need to be re-derived for
verification purposes anyway.
This performs various improvements to the naming of the branches of
the TxID and Signature Digests, and the Authorizing Data Commitment, as
initially suggested in the review of
QED-it/zcash-test-vectors#24.

(adapted from commit 8662fd6)
Rationale: these are long enough as it is, and ZSAs are an augmentation
to the existing Orchard pool, not a new pool of funds.
Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
clean up some formatting.

Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
Co-authored-by: Vivek Arte <46618816+vivek-arte@users.noreply.github.com>
@vivek-arte vivek-arte marked this pull request as draft March 11, 2025 18:33
Copy link

netlify bot commented Mar 11, 2025

Deploy Preview for zcash-zips-qedit ready!

Name Link
🔨 Latest commit 3d6f118
🔍 Latest deploy log https://app.netlify.com/sites/zcash-zips-qedit/deploys/67d081f5a7380e00082d7317
😎 Deploy Preview https://deploy-preview-101--zcash-zips-qedit.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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.

3 participants