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

Håndter at personer kan ha flere identer når vi starter, avslutter og henter periode #693

Merged
merged 4 commits into from
Mar 9, 2023

Conversation

marenger
Copy link
Contributor

@marenger marenger commented Mar 9, 2023

No description provided.

En person som har byttet ident (feks fra D-nr til fødselsnummer), kan ha startet en periode på sin tidligere ident. Når denne personen skal meldes ut av arbeidssøkerregisteret, må vi hente alle perioder for denne personens identer og finne den aktive perioden uavhengig av ident. Så avslutter vi denne perioden selv dersom den står oppført på tidligere fnr.
@marenger marenger requested a review from a team as a code owner March 9, 2023 11:01
@maccyber
Copy link
Contributor

maccyber commented Mar 9, 2023

Ser veldig bra ut 🙌

Usikker om vi bør oppdatere fødselsnummeret når vi avslutter en periode som har et annet gjeldende fødselsnummer enn den aktive perioden.

@marenger
Copy link
Contributor Author

marenger commented Mar 9, 2023

Usikker om vi bør oppdatere fødselsnummeret når vi avslutter en periode som har et annet gjeldende fødselsnummer enn den aktive perioden.

Ja, jeg også. Tror ikke det har noen praktisk betydning så lenge vi henter perioder på alle fødselsnumre, så jeg kan få dette ut, la todoen stå også kan vi diskutere det i fellesskap 😄

@marenger marenger merged commit 6e71529 into main Mar 9, 2023
@marenger marenger deleted the perioder-flere-fnr branch March 9, 2023 12:52
@maccyber
Copy link
Contributor

maccyber commented Mar 9, 2023

Høres ut som en plan

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.

None yet

2 participants