Time and Deadlines: Code review request

Code review request
English: Write to your team. You are done with your part of the code. Ask who has time to read and check it before the day ends.
Svenska: Skriv till ditt team. Du är klar med din del av koden. Fråga vem som har tid att läsa och granska den innan dagen är slut.

Hej #backend-team!

Hej #backend-team!

Alex här från NordTech Solutions AB i Stockholm — jag skriver till teamet för att säga att jag är klar med min del av koden för ticket NTS-2952.

PR är uppe på GitHub som draft och jag vill merge innan sprinten tar slut på fredag, men code review måste ske idag om vi ska hinna med regression i staging.

Vem har tid att läsa och granska den innan dagen är slut? Jag är flexibel med review-format: kommentarer i PR, parprogrammering eller snabb synk i Teams.

Vad jag levererat: backend-endpoints för export av veckostatistik till CSV och PDF i rapportmodulen som Ventilation Nord AB bad om i förra sprinten.

Scope: tre nya endpoints i Python-tjänsten, migration i PostgreSQL för export-logg, pytest-tester och uppdaterad OpenAPI-spec enligt vår mall.

Branch: feature/nts-2952-report-export · cirka fyrahundrade rader netto efter borttag av duplicerad logik som Markus Berg hjälpte mig identifiera igår.

Alla unit tests och integrationstester passerar lokalt med Docker Compose; CI är grön på senaste push klockan 11.40.

Markus Berg, du granskade arkitekturutkastet i förra veckan — om du har trettio minuter idag vore din review på affärslogik och query-prestanda ovärderlig.

Sofia Nilsson, inget deployment-kritiskt i PR:en, men kolla gärna om export-jobbet påverkar worker-kapacitet när flera användare exporterar samtidigt.

Anna Berg behöver merged kod till staging innan kunddemo torsdag — därför frågar jag specifikt om någon kan granska innan vi stänger dagen.

PR-beskrivningen följer vår mall: syfte, hur testa lokalt, risker, screenshots från staging och länk till Jira med acceptance criteria.

Jag har markerat två zoner i koden där jag vill ha extra ögon: batch-export över stora datumintervall och felhantering när PDF-generatorn timeoutar.

Code review innan dagen är slut underlättar att Sofia kan deploya till staging imorgon bitti utan att vi sitter med merge-konflikter kvällen innan demo.

Om du tar review: förväntad tid cirka tjugo till fyrtio minuter beroende på om du vill köra tester lokalt eller bara läsa diff i GitHub.

Jag är tillgänglig hela eftermiddagen för frågor — pinga i denna tråd eller i PR om något är oklart i validering eller exportformat.

Kort teknisk sammanfattning för den som skumma först: läs README-uppdatering i PR och sektionen Testplan i beskrivningen.

Inga breaking changes mot befintliga API:er; nya routes under /api/reports/export med samma auth som övriga rapportendpoints.

Jag har lagt till rate limiting enligt Markus förslag så att export inte kan missbrukas som DoS mot databasen.

Migrationen är reversibel; downgrade testad lokalt — Sofia bad om det explicit efter incidenten förra månaden.

Om ingen senior har tid idag: även review från kollega på medelnivå hjälper — jag vill inte att Markus blir ensam gatekeeper varje gång.

Deadline idag: jag hoppas på minst en godkännare och en kommentar innan 17.00 så jag kan fixa feedback i kväll om det behövs.

Efter godkänd review mergear jag själv och meddelar här när staging är uppdaterad — du behöver inte följa hela kedjan om du bara granskar.

Tack för att ni tar code review seriöst på NordTech — det är därför vi kan leverera till Ventilation Nord med relativt låg teknisk skuld.

PS: PR-länk i tråd nedan om Slack inte expanderar GitHub automatiskt — https://github.com/nordtech/platform/pull/1842 (intern repo).

PPS: Om du vill reviewa men bara har femton minuter, börja med filen export_service.py och test_export_batch.py — kärnan sitter där.

Standup imorgon: jag rapporterar om review status — hjälp mig undvika att säga vi väntar på review om någon redan kan ta det idag.

Jag är klar med min del av koden enligt ticket; det som återstår är granskning, merge och Sofia deployment — teamets tidsplan hänger på review idag.

Markus, du sa på måndag att små PR:er reviewas snabbare — jag delade upp export i två commits så diffen är lättare att läsa.

Sofia, worker-konfigurationen för export-jobb ligger i samma PR men i separat fil så du kan hoppa direkt till deployment-relaterade rader.

För kollegor som undrar varför idag: sprintdeadline fredag plus kunddemo torsdag gör att varje dag utan review kostar oss buffer.

Jag dokumenterar export-API i Confluence efter merge enligt rutin — review idag gör att dokumentationen kan följa samma vecka som koden.

Om du granskar: använd vår checklista i Confluence under Code review — läsbarhet, tester, säkerhet, prestanda, docs.

Ingen rush att godkänna utan att läsa — jag vill bara veta vem har tid så att vi inte sitter tysta till klockan 16.45.

Eftermiddag passar bäst för mig om du vill synka live: 13.30 till 16.00 jag har blockat review-tid i kalendern och svarar inom minuter.

Tack för att du läser — fråga vem har tid att läsa och granska innan dagen är slut är bättre än att anta att någon automatiskt ser PR-notisen.

Hör av er med emoji i tråden om du tar review: ögat räcker så jag ser att någon är på det innan jag går till fika.

Om två personer har tid är dubbel review välkommen på batch-export-delen — det är den mest riskfyllda ändringen tekniskt.

Jag är klar med kod, tester och PR-text — nu behöver jag teamets hjälp med granskning så att tidsplanen för sprint och demo håller.

Vi ses i tråden eller vid skrivbordet — jag har laptop med lokal miljö igång om du vill se export live under review.

Alex Kowalski, backend-utvecklare, NordTech Solutions AB, Stockholm.

/Alex