Skip to content

Conversation

lindeav
Copy link
Member

@lindeav lindeav commented Oct 19, 2026

Beskrivelse

Samme litt kronglete løsning fra i stad, men for at det som kjører på VM er oppdatert med det jeg trenger uten at vi merger alle endringer vi har på dev lager jeg to pull requests. En for main og en for dev slik at dev er oppdatert. Kopierer inne beskrivelser fra main sin pull request under.

Beskrivelse fra main sin pr:
For å kunne oppdatere prisene osv på produkter må vi endre litt på oppsettet vårt. Vi henter allerede info om når produktet sist ble oppdatert så vi trenger ikke å endre noe på databasen vi må kun endre på import og seed scriptet slik at vi ikke henter all informasjonen på nytt hver dag.

Forklar hovedhensikten og problemet den løser
Siden vi skal ha en nettside hvor man kan søke på ulike produkter og sammenligne priser fra ulike kjeder må vi oppdatere prisen fortløpende.

Relaterte issues

#60

Inkluder alle issues som denne PR stenger

Stenger ingen (cron-oppsettet på VM-en gjenstår).

Endringer gjort

  • import.ts: sorterer på dato (sort=date_desc) og stopper når den når varer eldre enn forrige import. Datoen leses fra data/last-import.txt.
  • import.ts og seed.ts: filstien kan settes med IMPORT_OUTPUT, så nattjobben ikke overskriver dumpen i git.
  • seed.ts: skriver ny dato til data/last-import.txt etter vellykket seeding.
  • seed.ts: beholder eksisterende kategori når en vare kommer inn uten kategori.
  • .gitignore: la til data/last-import.txt.

Hva slags type fiks inneholder denne PR

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

Sjekkliste

  • Testet lokalt
    Testet lokalt med last-import.txt satt til 18. oktober: hentet 33 sider (2392 varer) i stedet for alle, og seedet uten feil.
  • Dokumentert endringer
  • Sjekket for feilmeldinger

@lindeav lindeav requested a review from a team as a code owner October 19, 2026 14:40
Copy link
Member

@taraa taraa left a comment

Choose a reason for hiding this comment

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

Good! Ser ryddig ut, og definisjonen av ferdig dekker det viktigste.

@ingrihka ingrihka merged commit bb01244 into dev Oct 20, 2026
@lindeav lindeav deleted the feat/nightly-import branch October 21, 2026 13:36
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants