Bare en sakid #6878
Bare en sakid #6878
Conversation
| import java.math.BigDecimal | ||
| import java.time.YearMonth | ||
|
|
||
| data class KravgrunnlagId( | ||
| val value: Long, | ||
| ) | ||
|
|
||
| data class SakId( | ||
| val value: Long, |
There was a problem hiding this comment.
Denne er litt skummel, mener jeg så i en eller annen sammenheng at disse blir lagret ned med value i databasen siden de ikke er annotert med @JsonValue. Det betyr at gamle tilbakekrevinger vil slite med denne endringen. Da må eksisterende data migreres i alle fall. Forøvrig enig at det er greit å holde seg til en type på disse tingene.
There was a problem hiding this comment.
Kan heller se på å endre common SakId.sakId til SakId.value?
| gen.writeObjectFieldStart("sakId") | ||
| gen.writeObjectField("value", value.sakId.sakId) | ||
| gen.writeEndObject() | ||
| gen.writeObjectField("sakId", value.sakId) |
There was a problem hiding this comment.
Slik ble den serialisert tidligere
{"sakId":{"value":1},"sakType":"BARNEPENSJON","fnr":{"value":"123456"},"utbetalingslinjer":[]}
Sånn blir den serialisert nå:
{"sakId":1,"sakType":"BARNEPENSJON","fnr":{"value":"123456"},"utbetalingslinjer":[]}
sebassonav
force-pushed
the
feature/bare-en-sakid
branch
from
1312c06 to
06cdc05
Compare
sebassonav
force-pushed
the
feature/bare-en-sakid
branch
from
06cdc05 to
6ba2532
Compare
No description provided.