docs(i18n): refresh issue 279 close-out status
This commit is contained in:
@@ -2,7 +2,7 @@
|
|||||||
|
|
||||||
## [Unreleased]
|
## [Unreleased]
|
||||||
### Docs
|
### 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
|
### i18n
|
||||||
- Shared da/en lobby i18n contract is wired across Django and Angular MVP flows via `shared/i18n/lobby.json`.
|
- Shared da/en lobby i18n contract is wired across Django and Angular MVP flows via `shared/i18n/lobby.json`.
|
||||||
|
|||||||
@@ -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.
|
3. en release-readiness checkliste for i18n, forankret i et verificeret snapshot af `main` ved reviewtidspunktet.
|
||||||
|
|
||||||
Repo-state ved review-opdatering:
|
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).
|
- `main` peger nu på merge commit `1bc4c27` (PR #283), og inkluderer også PR #282 via merge commit `6ad5430`.
|
||||||
- Denne note er et snapshot af repo-tilstanden verificeret 2026-03-13 UTC, ikke en løbende garanti for senere `main`-ændringer.
|
- Denne note er opdateret mod repo-tilstanden verificeret 2026-03-13 UTC, ikke en løbende garanti for senere `main`-ændringer.
|
||||||
- Åbne i18n-relaterede PR'er uden for `main`:
|
- Der er ingen åbne release-afklaringer tilbage for PR #282/#283; begge er allerede landet på `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)
|
|
||||||
|
|
||||||
## Current i18n MVP state on `main`
|
## Current i18n MVP state on `main`
|
||||||
|
|
||||||
@@ -99,7 +97,7 @@ Kort version til annoterede release-notes:
|
|||||||
|
|
||||||
## Release-readiness checklist for i18n
|
## 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
|
### 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] Backend artifact for issue #225 findes.
|
||||||
- [x] Frontend/shared loader artifact for issue #257 findes.
|
- [x] Frontend/shared loader artifact for issue #257 findes.
|
||||||
- [x] Drift-check dokumentation findes i `docs/i18n-drift-check.md`.
|
- [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
|
### 3) Code readiness on current branch topology
|
||||||
|
|
||||||
- [x] Angular MVP host/player i18n flow er på `main` (PR #281).
|
- [x] Angular MVP host/player i18n flow er på `main` (PR #281).
|
||||||
- [x] Shared locale/bootstrap wiring er på `main`.
|
- [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).
|
- [x] Django i18n hardening fra issue #275 er på `main` via PR #283 (merge commit `1bc4c27`).
|
||||||
- [ ] PR #283 er stadig en separat release-afklaring, selv om den aktuelt er mergebar i Gitea.
|
- [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”
|
### 4) Release gate before shipping i18n as “done”
|
||||||
|
|
||||||
- [ ] Merge eller afvis eksplicit PR #282, så parity-artifact-status er afklaret.
|
- [x] PR #282 er allerede merged, så parity-artifact-status er afklaret på `main`.
|
||||||
- [ ] Merge eller afvis eksplicit PR #283, så backend hardening-status er afklaret.
|
- [x] PR #283 er allerede merged, så backend hardening-status er afklaret på `main`.
|
||||||
- [ ] Kør drift-check fra repo root:
|
- [ ] Kør drift-check fra repo root:
|
||||||
```bash
|
```bash
|
||||||
python3 scripts/check_i18n_drift.py
|
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
|
## 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) 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.
|
||||||
- **PR #282** (parity artifact)
|
|
||||||
- **PR #283** (Django i18n hardening, stadig en åben release-afklaring selv om den var `mergeable: true` ved verifikation)
|
|
||||||
|
|
||||||
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.
|
||||||
|
|||||||
Reference in New Issue
Block a user