Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Legg til kolonne for inntektsmelding-ID i database #849

Merged
merged 1 commit into from
Mar 21, 2025
Merged

Conversation

bjerga
Copy link
Contributor

@bjerga bjerga commented Mar 21, 2025

UNIQUE skal også sørge for at kolonnen blir indeksert, som vi trenger når vi skal oppdatere radene med journalpost-ID.

Sniker med kolonne for avsendernavn også, som nå lagres i IM på gammelt format.

@bjerga bjerga requested a review from a team as a code owner March 21, 2025 12:36
@mortenbyhring
Copy link
Contributor

avsendernavn - altså det som heter innsenderNavn i den gamle IM? Det er vel navn på personen som sender inn,
er det egentlig noen grunn til at vi vil ha det i en egen kolonne, og trenger vi å lagre det i det hele tatt?

@bjerga
Copy link
Contributor Author

bjerga commented Mar 21, 2025

@mortenbyhring

avsendernavn - altså det som heter innsenderNavn i den gamle IM?

Stemmer. avsender.navn i nytt format.

Det er vel navn på personen som sender inn, er det egentlig noen grunn til at vi vil ha det i en egen kolonne, og trenger vi å lagre det i det hele tatt?

Det vises i kvitteringen, så vi må lagre det. Legger det i en egen kolonne for å frigjøre kvitteringen fra den lagrede inntektsmelding, siden det er det eneste feltet kvitteringen er avhengig av derfra.

@mortenbyhring
Copy link
Contributor

@mortenbyhring

avsendernavn - altså det som heter innsenderNavn i den gamle IM?

Stemmer. avsender.navn i nytt format.

Det er vel navn på personen som sender inn, er det egentlig noen grunn til at vi vil ha det i en egen kolonne, og trenger vi å lagre det i det hele tatt?

Det vises i kvitteringen, så vi må lagre det. Legger det i en egen kolonne for å frigjøre kvitteringen fra den lagrede inntektsmelding, siden det er det eneste feltet kvitteringen er avhengig av derfra.

Aha, skjønner. Trodde vi kanskje bare slo opp navnet på avsender on the fly, men uansett ingen big deal.

@bjerga
Copy link
Contributor Author

bjerga commented Mar 21, 2025

Aha, skjønner. Trodde vi kanskje bare slo opp navnet på avsender on the fly, men uansett ingen big deal.

@mortenbyhring Vi lagrer ikke fnr på avsender, så det er derfor vi ikke slår opp on the fly.

@bjerga bjerga merged commit b92db42 into main Mar 21, 2025
18 checks passed
@bjerga bjerga deleted the db-kolonne-im-id branch March 21, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants