Skip to content

feat(prot): replace block index with account [DO-41]

Ethan Reesor requested to merge DO-41-block-account into develop

Closes DO-41. For BVN X block N, create bvn-${X}.acme/ledger/${N} (and for the DN). This account records (account, chain, index) for every entry added to a chain during the block. This replaces the block chain updates index, which is not persisted in snapshots and therefore may be lost.

This data is not strictly necessary for the protocol, but it would be tedious to rebuild if it were lost. Creating an account for each block seems like the most natural way to implement this, since we do not have any sub-account like mechanism.

Review Checklist

If any item is not complete, the merge request is not ready to be reviewed and must be marked Draft:.

  • The merge request title is in the format <change type>(<change scope>): <short description> [<task id>]
    • For example, feat(cli): add QR code generation [AC-123]
    • For details, see CONTRIBUTING.md
  • The description includes Closes <jira task ID> (or rarely Updates <jira task ID>)
  • The change is fully validated by tests that are run during CI
    • In most cases this means a test in "validate.sh"
    • In some cases, a Go test may be acceptable
    • Validation is not applicable to things like documentation updates
    • Purely UI/UX changes can be manually validated, such as changes to human-readable output
    • For all other changes, automated validation tests are an absolute requirement unless a maintainer specifically explains why they are not in a comment on this merge request
  • The change is marked with one of the validation labels

Merge Checklist

  • CI is passing
  • Merge conflicts are resolved
  • All discussions are resolved

Related to DO-41

Merge request reports