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

Document consensus rules from Zcash spec: 7.3 Spend Description Encoding and Consensus #3224

Closed
Tracked by #3125
mpguerra opened this issue Dec 14, 2021 · 0 comments · Fixed by #3575
Closed
Tracked by #3125

Document consensus rules from Zcash spec: 7.3 Spend Description Encoding and Consensus #3224

mpguerra opened this issue Dec 14, 2021 · 0 comments · Fixed by #3575
Assignees
@conradoplg

Comments

@mpguerra
Copy link
Contributor

mpguerra commented Dec 14, 2021 •
edited
Loading

For each consensus rule:

  • check that the rule is implemented in Zebra
  • make sure the latest version of the rule is quoted in the docs for the code that implements it
  • make sure the code implements the latest version of the rule

Consensus Rule

https://zips.z.cash/protocol/protocol.pdf#spendencodingandconsensus

Screenshot 2021-12-16 at 15 58 27

Implementation
@mpguerra mpguerra mentioned this issue Dec 14, 2021
Closed
50 tasks
@mpguerra mpguerra changed the title Document 7.3 Spend Description Encoding and Consensus consensus rule Document consensus rules from Zcash spec: 7.3 Spend Description Encoding and Consensus Dec 16, 2021
@conradoplg conradoplg mentioned this issue Feb 14, 2022
Closed
@conradoplg conradoplg assigned conradoplg and unassigned dconnolly Feb 14, 2022
@ftm1000 ftm1000 added the S-needs-triage Status: A bug report needs triage label Feb 16, 2022
@conradoplg conradoplg mentioned this issue Feb 17, 2022
Merged
3 tasks
@mergify mergify bot closed this as completed in #3575 Feb 19, 2022
@mpguerra mpguerra removed the S-needs-triage Status: A bug report needs triage label Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@conradoplg conradoplg

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Document consensus rules from 7.3 Spend Description Encoding and Consensus ZcashFoundation/zebra
4 participants
@conradoplg @dconnolly @mpguerra @ftm1000