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.5 Action Description Encoding and Consensus #3226

Closed
Tracked by #3125
mpguerra opened this issue Dec 14, 2021 · 1 comment · Fixed by #3549
Closed
Tracked by #3125

Document consensus rules from Zcash spec: 7.5 Action Description Encoding and Consensus #3226

mpguerra opened this issue Dec 14, 2021 · 1 comment · Fixed by #3549
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#actionencodingandconsensus

Screenshot 2021-12-16 at 15 48 20

Implementation
@mpguerra mpguerra mentioned this issue Dec 14, 2021
Closed
50 tasks
@mpguerra mpguerra changed the title Document 7.5 Action Description Encoding and Consensus consensus rule Document consensus rules from Zcash spec: 7.5 Action Description Encoding and Consensus Dec 16, 2021
@conradoplg conradoplg assigned conradoplg and unassigned dconnolly Feb 14, 2022
@conradoplg conradoplg mentioned this issue Feb 16, 2022
Merged
3 tasks
@ftm1000
Copy link

ftm1000 commented Feb 16, 2022

Hey team! Please add your planning poker estimate with ZenHub @gustavovalverde @dconnolly @conradoplg @jvff @oxarbitrage @teor2345 @upbqdn

@ftm1000 ftm1000 added S-needs-triage Status: A bug report needs triage and removed S-needs-triage Status: A bug report needs triage labels Feb 16, 2022
@mergify mergify bot closed this as completed in #3549 Feb 21, 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 4.6 Action Descriptions ZcashFoundation/zebra
4 participants
@conradoplg @dconnolly @mpguerra @ftm1000