Puzzlepart / prosjektportalen365

Prosjektportalen for Microsoft 365
MIT License
29 stars 15 forks source link

Rettet et problem hvor det ikke var mulig å lagre tom verdi i valuta/tall felt fra redigeringspanel #1512

Closed stigre closed 2 months ago

stigre commented 2 months ago

Pull request (PR)

Sørg for at du ber om PR for din branch (høyre side). Sørg for at du gjør en PR mot riktig release-branch (venstre side). Sjekk commits og alle commit-meldingene.

Sjekklisten din

Alle sjekkpunktene under må være sjekket av og godkjent for at vi skal kunne merge branchen din mot dev.

Beskrivelse

Verdi fra valuta/tall felt er av type string. Hvis feltet tømmes av bruker forsøker den å lagre en tom streng - som feiler. Denne endringen setter null dersom verdien fra feltet er en tom streng som lar deg lagre den tomme verdien.

Hvordan teste

  1. Gå til Statusrapport for et prosjekt
  2. Opprett statusrapport dersom den ikke finnes fra før
  3. Legg inn verdi i et valuta/tall felt hvis ingen har verdier og lagre
  4. Fjern verdier i valuta/tall feltene og forsøk å lagre
  5. Kontroller at feltet ble lagret med tom verdi (forsvinner fra rapporten)

Relevante issues (hvis aktuelt)

Sjekkliste for godkjenner

Alle sjekkpunktene under må være sjekket av og godkjent av reviewers for at vi skal kunne merge branchen din mot dev.

olemp commented 2 months ago

RØDDIG @stigre!

tarjeieo commented 2 months ago

Dette er en commit jeg kan stille meg bak! 🔥

trondoe commented 2 months ago

Så vakkert at det frembringer tårer i øyekroken!

okms commented 2 months ago

Du er committed Stian! Dette ser bra ut!

pzlespen commented 2 months ago

Wow, hvorfor kan ikke alle bare være som Stian? GOAT

olemp commented 2 months ago

Stian

image

Alle oss andre

image

Remi749 commented 2 months ago

@stigre right now: image

damsleth commented 2 months ago

10x DEVELOPER! 💫

siifux commented 2 months ago

@stigre Dette er jo bare helt njuuuuuudelig!!!! Good job!

Fikk fanget et bilde av deg i øyeblikket du sendte PR-en inn:

image