Fjern overflødig strengverdi i enum for json-nøkler #796
Conversation
| BEHOV -> "@behov" | ||
| LOESNING -> "@løsning" | ||
| FORESPOERSEL_ID -> "forespoerselId" | ||
| SPINN_INNTEKTSMELDING_ID -> "spinnInntektsmeldingId" | ||
| else -> name.lowercase() |
There was a problem hiding this comment.
Luring den her 😅
Er det noen av disse vi klarer å bytte ut også, eller er camel case og krøllalfa-case så hardt innarbeidet på andre siden av Pri at det ikke er noe poeng?
There was a problem hiding this comment.
Vi kan egentlig endre alle. Vi må bare gidde å gjøre jobben.
| EVENT_NAME -> "@event_name" | ||
| BEHOV -> "@behov" |
There was a problem hiding this comment.
Hva signaliserer alfakrøllen i nøklene egentlig? At det er rapids and rivers-nøkler?
There was a problem hiding this comment.
RR-nøklene bruker @-prefixen, og disse to er innarbeidede nøkler. Vi startet å bruke dem, men har siden den gang ikke hatt noe særlig forhold til den "standarden". Vi kan nok endre disse også, men det er noen funksjoner i bomlo-cli som støtter "@event_name" out of the box, som vi da må fikse litt på for at det skal fungere.
Med strengene så hadde vi en slags dobbel definisjon. Nå som vi sammenstilt strengverdi med enumnavn de fleste tilfellene, så kan vi kvitte oss med den unødvendige strengen.