redrossa / adfire

Accounting platform for achieving FIRE
2 stars 0 forks source link

As Redrossa, I can format my input files, which will insert all optional columns into them #24

Open redrossa opened 3 weeks ago

redrossa commented 3 weeks ago

Description

The current workflow on the integrity verification with the checksum file is bugged. When optional but essential columns like the ID is omitted in input records, the checksum produced fills in the ID, which errors out subsequent formatting because the input record still has omitted IDs, while the checksum doesn't, which evaluates to an invalid checksum. Furthermore, the checksum file produced as input for subsequent formatting puts an extra burden on the user workflow.

A proposed solution to address these issues are:

  1. Formatting does not produce checksum file anymore, instead it edits the input files directly with the fully filled and correct values and columns.
  2. A new column is added to store hash of each posted entry, this is supposed to not be edited by users
  3. The integrity of a record is checked so that each row's hash equals the stored hash

Acceptance Criteria

redrossa commented 1 week ago

Definitions in a Record

redrossa commented 1 week ago

Properties of a Record