From 23bef24e18cbdd8e7cabc02d3d687a2649ef8f72 Mon Sep 17 00:00:00 2001 From: Erol Haagenrud Date: Sat, 8 Aug 2026 16:45:24 +0200 Subject: [PATCH] Dokumenter slag-liste-funksjonen i CHANGELOG.md Punkt 50. Se forrige commit for koden. --- CHANGELOG.md | 56 ++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 56 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index f79f3dd..6f19019 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -9332,3 +9332,59 @@ Neste steg: identisk start-/sluttkoordinat) at en avvist 0m-innsending nå viser feilmeldingen i arket i stedet for å lukke seg stille. Scratch-miljøet ryddet opp fullstendig. + +50. **Slag-for-slag GPS-avstandsmåling: liste over egne slag + fant en + TREDJE stille-feil-bug — 2026-08-08, brukeroppfølging.** Brukeren + prøvde på nytt etter punkt 49s fiks (bekreftet: målte nå 13 meter, + ikke 0), men spurte "jeg kan ikke se det delte slaget i ettertid?". + + **Diagnose (skrivebeskyttede spørringer mot ekte `teecup_db` + ekte + API-logger):** slaget lå riktig i `round_shot` (13m, ikke 0 -- punkt + 49s fiks virket), men `shared_round_message_id` var tom, og + `round_message`-tabellen hadde null rader for runden. API-loggene + viste at `POST .../shots` ga 201 Created, men det fantes INGEN + etterfølgende kall til del-endepunktet i det hele tatt. Konklusjon: + "Del i feeden"-valget var ikke aktivt idet brukeren trykket "Lagre + slag" (arket starter forfra, inkl. tilbake til "Behold privat" + -standardvalget, hver gang det åpnes på nytt -- forklart til + brukeren). + + Brukerens motspørsmål var det egentlig viktige: **et privat, ikke-delt + slag burde uansett kunne SES i ettertid** -- merkelappen viste + tidligere kun et tall ("N slag målt"), aldri kølle/avstand for de + faktiske slagene. Dette var en reell mangel i inngangspunkt 2 (punkt + 46), ikke bare en brukerforvirring. + + **Bygget:** `ShotMeasurementEntry` lagrer nå hele slag-listen (ikke + bare `.length`), med en ny utvidbar `ShotList` (kølle + avstand per + slag, "Delt"-merke eller en "Del"-lenke for et ikke-delt slag, en + slette-knapp per slag via det allerede eksisterende + `DELETE /rounds/{id}/shots/{shot_id}`-endepunktet). Splittet den + tidligere ene knappen i to: en utvidbar "N slag målt"-disclosure + (kun synlig når `count > 0`) og en separat, alltid synlig "+"-knapp + for å måle et NYTT slag -- unngår at å åpne listen og å starte en ny + måling er samme handling. + + **Tredje stille-feil-bug funnet og rettet i samme runde:** akkurat + som punkt 49s rotårsak 2, sjekket heller ikke re-del-fra-liste-kallet + (`shareExisting`) svaret sitt. Rettet parallelt med bygningen av + listen, IKKE etter en ny brukerrapport denne gangen. En mislykket + deling (enten ved førstegangs innsending eller re-del fra listen) + vises nå som en egen, kortvarig feiltekst ved siden av merkelappen + (`shareError`-state) -- BEVISST atskilt fra `submitError` (som + fortsatt holder selve MÅLE-arket åpent ved en avvist innsending): + siden selve slaget alerede er lagret på tidspunktet en delingsfeil + kan oppstå, ville gjenbruk av `submitError` (som holder arket åpent) + risikert at brukeren trykker "Lagre slag" på nytt og oppretter et + duplikat-slag. + + `tsc --noEmit` rent. **Scratch-verifisert i ekte nettleser** (tjuende + scratch-miljø denne økten, samme migrasjon-002-forsiktighet, + `teecup_db` bekreftet uendret): opprettet to slag direkte mot API-et + (Driver 215,5m, 7-jern 145m), bekreftet merkelappen viser "2 slag + målt" som en egen utvidbar knapp ved siden av en separat "+"-knapp, + utvidet listen og så begge slag med riktig kølle/avstand, klikket + "Del" på Driver-slaget og bekreftet det ble til "Delt" OG at meldingen + faktisk postet riktig tekst til `/rounds/{id}/messages`, slettet + 7-jern-slaget og bekreftet det forsvant fra listen ("1 slag målt" + etterpå). Scratch-miljøet ryddet opp fullstendig.