zebra
zebra copied to clipboard
add(chain): Impl `zcash_protocol::consensus::Parameters` for `Network`
Motivation
The new scanning API in zcash_client_backend
requires the zcash_protocol::consensus::Parameters
trait for the passed network. This is a preparatory PR for the transition to the new API.
PR Author Checklist
Check before marking the PR as ready for review:
- [x] Will the PR name make sense to users?
- [x] Does the PR have a priority label?
- [ ] Have you added or updated tests?
- [x] Is the documentation up to date?
Solution
- Use full paths for types from zcash_primitives
- Impl and use conversion between network kinds
- Impl conversions between network upgrades
- Impl zcash_protocol::consensus::Parameters
Reviewer Checklist
Check before approving the PR:
- [ ] Does the PR scope match the ticket?
- [ ] Are there enough tests to make sure it works? Do the tests cover the PR motivation?
- [ ] Are all the PR blockers dealt with? PR blockers can be dealt with in new tickets or PRs.
And check the PR Author checklist is complete.
@Mergifyio refresh
refresh