From 4a12cee6ee8c877d5cba008699751f379deb2564 Mon Sep 17 00:00:00 2001 From: Asger Geel Weirsoee Date: Fri, 13 Mar 2026 11:11:12 +0000 Subject: [PATCH] docs(i18n): refresh issue 279 close-out status --- CHANGELOG.md | 2 +- docs/ISSUE-279-I18N-MVP-CLOSEOUT.md | 28 ++++++++++++---------------- 2 files changed, 13 insertions(+), 17 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index d9a6956..9cc3221 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,7 +2,7 @@ ## [Unreleased] ### Docs -- Added `docs/ISSUE-279-I18N-MVP-CLOSEOUT.md` with i18n MVP close-out guidance covering migration impact, releasable changelog text, and a release-readiness checklist anchored to `main` snapshot reviewed on 2026-03-13 UTC. +- Added `docs/ISSUE-279-I18N-MVP-CLOSEOUT.md` with i18n MVP close-out guidance covering migration impact, releasable changelog text, and a release-readiness checklist refreshed against `main@1bc4c27` after PR #282/#283 landed on 2026-03-13 UTC. ### i18n - Shared da/en lobby i18n contract is wired across Django and Angular MVP flows via `shared/i18n/lobby.json`. diff --git a/docs/ISSUE-279-I18N-MVP-CLOSEOUT.md b/docs/ISSUE-279-I18N-MVP-CLOSEOUT.md index 9ce8d97..c28acf0 100644 --- a/docs/ISSUE-279-I18N-MVP-CLOSEOUT.md +++ b/docs/ISSUE-279-I18N-MVP-CLOSEOUT.md @@ -11,11 +11,9 @@ Dette dokument lukker MVP-sporet for issue #175 med tre konkrete ting: 3. en release-readiness checkliste for i18n, forankret i et verificeret snapshot af `main` ved reviewtidspunktet. Repo-state ved review-opdatering: -- `main` peger på merge commit `e6ca18f` (`test: issue #278 da+en smoke gate and primary-only audio verification`, PR #285). -- Denne note er et snapshot af repo-tilstanden verificeret 2026-03-13 UTC, ikke en løbende garanti for senere `main`-ændringer. -- Åbne i18n-relaterede PR'er uden for `main`: - - PR #282 `docs(#277): add shared i18n parity artifact` (`mergeable: true` ved verifikation) - - PR #283 `feat(#275): harden django i18n locale negotiation and fallback` (`mergeable: true` ved verifikation) +- `main` peger nu på merge commit `1bc4c27` (PR #283), og inkluderer også PR #282 via merge commit `6ad5430`. +- Denne note er opdateret mod repo-tilstanden verificeret 2026-03-13 UTC, ikke en løbende garanti for senere `main`-ændringer. +- Der er ingen åbne release-afklaringer tilbage for PR #282/#283; begge er allerede landet på `main`. ## Current i18n MVP state on `main` @@ -99,7 +97,7 @@ Kort version til annoterede release-notes: ## Release-readiness checklist for i18n -Status er vurderet mod verificeret snapshot `main@e6ca18f` (reviewet 2026-03-13 UTC). +Status er vurderet mod verificeret snapshot `main@1bc4c27` (reviewet 2026-03-13 UTC, inkl. PR #282/#283). ### 1) Shared contract and locale behavior @@ -116,19 +114,19 @@ Status er vurderet mod verificeret snapshot `main@e6ca18f` (reviewet 2026-03-13 - [x] Backend artifact for issue #225 findes. - [x] Frontend/shared loader artifact for issue #257 findes. - [x] Drift-check dokumentation findes i `docs/i18n-drift-check.md`. -- [ ] Parity artifact fra issue #277 er ikke på `main` endnu (åben som PR #282, `mergeable: true` ved verifikation). +- [x] Parity artifact fra issue #277 er på `main` via PR #282 (merge commit `6ad5430`). ### 3) Code readiness on current branch topology - [x] Angular MVP host/player i18n flow er på `main` (PR #281). - [x] Shared locale/bootstrap wiring er på `main`. -- [ ] Django i18n hardening fra issue #275 er ikke på `main` endnu (åben som PR #283, `mergeable: true` ved verifikation). -- [ ] PR #283 er stadig en separat release-afklaring, selv om den aktuelt er mergebar i Gitea. +- [x] Django i18n hardening fra issue #275 er på `main` via PR #283 (merge commit `1bc4c27`). +- [x] PR #283 er ikke længere en separat release-afklaring; hardeningen er allerede indarbejdet på `main`. ### 4) Release gate before shipping i18n as “done” -- [ ] Merge eller afvis eksplicit PR #282, så parity-artifact-status er afklaret. -- [ ] Merge eller afvis eksplicit PR #283, så backend hardening-status er afklaret. +- [x] PR #282 er allerede merged, så parity-artifact-status er afklaret på `main`. +- [x] PR #283 er allerede merged, så backend hardening-status er afklaret på `main`. - [ ] Kør drift-check fra repo root: ```bash python3 scripts/check_i18n_drift.py @@ -162,10 +160,8 @@ Status er vurderet mod verificeret snapshot `main@e6ca18f` (reviewet 2026-03-13 ## Close-out conclusion -**Konklusion:** i18n-MVP'en er i praksis implementeret på `main`, og issue #279 kan levere den manglende release-/migration-closeout dokumentation uden nye kodeændringer i app-logikken. +**Konklusion:** i18n-MVP'en er implementeret på `main`, og issue #279 leverer den manglende release-/migration-closeout dokumentation uden nye kodeændringer i app-logikken. -Men hvis i18n-sporet skal kaldes fuldt release-ready, mangler der stadig en eksplicit beslutning om de to åbne opfølgnings-PR'er: -- **PR #282** (parity artifact) -- **PR #283** (Django i18n hardening, stadig en åben release-afklaring selv om den var `mergeable: true` ved verifikation) +PR #282 (parity artifact) og PR #283 (Django i18n hardening) er nu begge merged på `main`, så close-out-noten, changelog-teksten og release-readiness-checklisten kan behandles som indbyrdes konsistente for det verificerede snapshot. -Det betyder, at close-out-noten er komplet, men selve release-gaten bør betragte #282/#283 som åbne afklaringer, ikke som allerede landede leverancer. +Det betyder, at de resterende release-gates for i18n nu er de almindelige verificeringstrin (drift-check, backend/frontend-smoke, migrations-konsistens, staging deploy og changelog-reference) — ikke længere afklaring af om #282/#283 skal lande.