ART M4: review — full paging, include/exclude, versioned cell edits (BR-25, BR-26, BR-27) #108

Activity

1. amar-python

   added this to the ART extension milestone on Oct 26, 2026
2. amar-python

   added

   artART framework extension workART framework extension work

   on Oct 26, 2026
3. amar-python commented on Oct 28, 2026

   

   amar-python

   on Oct 28, 2026 – with Claude

   OwnerAuthor

   More actions

   Decisions by the product owner, 28 Oct 2026

   1. Review is recorded now and applied in M6. M4 records include and exclude decisions and cell edits against the stored upload (M3's source_rows), and everyone in the tenant can see them. The table imported at upload time doesn't change. M6's approval and staged import will import the reviewed version.
   2. Who reviews: reviewer and tenant_admin may decide rows and edit cells. Everyone with read_rows can view.
   3. A reason is required for every decision and edit. The API refuses an empty one.

   Parts

   • M4a (API and database):
     • Insert-only row_decisions and cell_edits, where the latest entry wins and the history is kept.
     • The column types each upload used are stored, so every row can be validated, not only the first 200 failures.
     • GET /api/csv/uploads/{id}/rows pages through every row, giving original and current values, validation errors and the row's decision.
     • Endpoints to record a decision and a cell edit, plus a row's history. Each change is also written to the audit log, with before and after values.
   • M4b (frontend): a review screen that pages through every row, with include and exclude, cell edits and the history.

   ---

   Generated by Claude Code
4. amar-python

   added 4 commits that reference this issue on Oct 28, 2026

   fix(api): allow PUT in CORS so the browser can send review changes (#108

   ...

   7470338

   test(review): positive and negative test cases for the review API (#108)

   ...

   ef8bd89

   Merge pull request #142 from amar-python/feat/108-m4a-review-api

   ...

   5a3df99

   Merge pull request #143 from amar-python/feat/108-m4b-review-screen

   ...

   4abefc8