solidprivacy-nl commited on
Commit
bf2d151
·
1 Parent(s): 5342e0e

Mark v12.5 app verification complete

Browse files
Files changed (1) hide show
  1. CHANGELOG.md +10 -4
CHANGELOG.md CHANGED
@@ -28,7 +28,7 @@ For UI/UX-only work, prefer pure helper modules and tests before touching Stream
28
 
29
  ## v12.5 — Final review summary
30
 
31
- Status: UI integration implemented; awaiting GitHub Actions, Hugging Face sync and app verification.
32
 
33
  Purpose:
34
 
@@ -45,6 +45,7 @@ Files added or changed:
45
  - `fix_streamlit_nested_expanders.py`
46
  - `WORKPACKAGES.md`
47
  - `CHANGELOG.md`
 
48
 
49
  Main changes:
50
 
@@ -63,17 +64,22 @@ Main changes:
63
  - Added Dutch readiness labels and markdown summary lines.
64
  - Integrated the summary into the existing startup UI patch so the app shows `Eindcontrole vóór download` immediately above the download section.
65
  - Kept the summary advisory only: it displays counts and readiness labels but does not block or alter downloads.
 
66
 
67
- Testing:
68
 
69
  - Added unit tests for `review_summary.py`.
70
  - Added a UI patch contract test to verify that the summary helper is imported and displayed before downloads.
71
  - Local targeted validation before UI integration passed: `PYTHONPATH=. pytest -q tests/test_review_summary.py` → 5 passed.
72
- - GitHub Actions and Hugging Face sync need confirmation for the UI integration commits.
 
 
 
73
 
74
  Intentionally not changed:
75
 
76
- - No direct edit to `presidio_streamlit.py`.
 
77
  - No recognizer changes.
78
  - No entity-type expansion.
79
  - No export/download blocking.
 
28
 
29
  ## v12.5 — Final review summary
30
 
31
+ Status: completed and app verified.
32
 
33
  Purpose:
34
 
 
45
  - `fix_streamlit_nested_expanders.py`
46
  - `WORKPACKAGES.md`
47
  - `CHANGELOG.md`
48
+ - `handover/workpackages/20260607_1345_v12_5_review_summary_closeout.md`
49
 
50
  Main changes:
51
 
 
64
  - Added Dutch readiness labels and markdown summary lines.
65
  - Integrated the summary into the existing startup UI patch so the app shows `Eindcontrole vóór download` immediately above the download section.
66
  - Kept the summary advisory only: it displays counts and readiness labels but does not block or alter downloads.
67
+ - Formally closed v12.5 after coordinator/user verification.
68
 
69
+ Testing and verification:
70
 
71
  - Added unit tests for `review_summary.py`.
72
  - Added a UI patch contract test to verify that the summary helper is imported and displayed before downloads.
73
  - Local targeted validation before UI integration passed: `PYTHONPATH=. pytest -q tests/test_review_summary.py` → 5 passed.
74
+ - Coordinator reported GitHub Actions tests green for the v12.5 review summary line.
75
+ - Coordinator reported GitHub to Hugging Face sync green for the v12.5 review summary line.
76
+ - Hugging Face app was visually verified and showed `Eindcontrole vóór download` before the download section.
77
+ - Downloads were reported as still working after verification: text, CSV, DOCX and PDF.
78
 
79
  Intentionally not changed:
80
 
81
+ - No direct edit to `presidio_streamlit.py` during closeout.
82
+ - No direct edit to `fix_streamlit_nested_expanders.py` during closeout.
83
  - No recognizer changes.
84
  - No entity-type expansion.
85
  - No export/download blocking.