Skip to content

Conversation

ingrihka
Copy link
Member

@ingrihka ingrihka commented Oct 2, 2026

Beskrivelse

Utforsket Kassalapp API for å se hvordan dataene faktisk ser ut før vi låser datamodellen. Funnene er grunnlaget for databaseskjemaet (#7) og importskriptet (#8).

Relaterte issues

Closes #6

Endringer gjort

  • docs/kassalapp.md: hvilke felt en vare har, datakvalitet, kategorier, merking, paginering, hvem som trenger API-nøkkel, og åpne spørsmål til task: Installer PostgreSQL lokalt og lag databaseskjema #7 og task: Importscript og seed-script #8
  • backend/src/scripts/explore-kassalapp.ts: script som henter /categories, /labels og 500 varer, og skriver ut statistikk over manglende felt, butikker, labels og kategorier. Kjøres med npx tsx src/scripts/explore-kassalapp.ts fra backend/

Viktigste funn

  • 4,6 % av varene mangler EAN og 11,8 % mangler kategori. Ingen mangler bilde eller pris
  • Samme EAN finnes én gang per kjede (Norvegia 500 g finnes i 7 kjeder), så vi lagrer én rad i products og én rad per kjede i prices
  • Økologisk = debio/euroleaf/krav, norskprodusert = nyt_norge. Laktosefri finnes ikke som merke og må utledes fra allergenet melk
  • De første sidene av /products er skjeve (mest barneprodukter og dessert), så importen bør hente per kategori
  • Bare den som kjører importen trenger API-nøkkel. Resten av gruppa seeder fra JSON-dumpen (task: Importscript og seed-script #8)

Hva slags type fiks inneholder denne PR

  • Bug fix
  • Ny funksjonalitet
  • Refactor
  • Dokumentasjon
  • Testing

Sjekkliste

  • Testet lokalt
  • Dokumentert endringer
  • Sjekket for feilmeldinger

Til reviewer

Skjermbilder

Ingen UI-endring.

@ingrihka ingrihka added documentation Improvements or additions to documentation setup labels Oct 2, 2026
@ingrihka ingrihka added this to the 2 - underveis innlevering milestone Oct 2, 2026
@ingrihka ingrihka self-assigned this Oct 2, 2026
@ingrihka ingrihka requested a review from a team as a code owner October 2, 2026 12:11
@lindeav lindeav self-requested a review October 2, 2026 13:00
Copy link
Member

@lindeav lindeav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dette ser bra ut! Jeg har også testet lokalt og fått de samme funnene. Godt jobba!!!!

Screenshot 2026-10-02 at 15 36 17

@lindeav lindeav merged commit cc9ed49 into main Oct 2, 2026
@lindeav lindeav deleted the setup/6-kassalapp-api branch October 2, 2026 14:14
Sign in to join this conversation on GitHub.
Labels
documentation Improvements or additions to documentation setup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

task: Skaff API-nøkkel og utforsk Kassalapp-dataene
2 participants