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

Utvidelse av nummerserie for D-nummer (validering) #55

Merged
merged 3 commits into from
Feb 16, 2022
Merged

Conversation

eivinhb
Copy link
Member

@eivinhb eivinhb commented Jan 14, 2022

ref.:
https://skatteetaten.github.io/folkeregisteret-api-dokumentasjon/sporsmal-og-svar/

Jeg leser det slik at det er kun D-nummer (ref. sitat i under) som skal
påvirkes av det nye regimet. Det er ny type logikk, og vi må derfor ha
forskjellige nummerserier i individnummer om det er D-nummer eller vanlig
tildelt nummer.

Jeg har lagt inn eksempler på syntetiske nummere samt "ekte" D-nummer.

ref.:https://skatteetaten.github.io/folkeregisteret-api-dokumentasjon/nyheter/
under "2021-06-24 - Utvidelse av D-nummerserien"
Sitat:

Konsumenter som benytter individnummeret til å utlede korrekt fødselsår
for D-nummerpersoner må endre sine systemløsninger. Endringen vil
produksjonssettes medio august 2021 og det vil kunne tildeles
D-numre med nye nummerserier umiddelbart.

ref.:
https://skatteetaten.github.io/folkeregisteret-api-dokumentasjon/sporsmal-og-svar/

Jeg leser det slik at det er kun D-nummer (ref. sitat i under) som skal
påvirkes av det nye regimet. Det er ny type logikk, og vi må derfor ha
forskjellige nummerserier i individnummer om det er D-nummer eller vanlig
tildelt nummer.

Jeg har lagt inn eksempler på syntetiske nummere samt "ekte" D-nummer.

ref.:https://skatteetaten.github.io/folkeregisteret-api-dokumentasjon/nyheter/
under "2021-06-24 - Utvidelse av D-nummerserien"
Sitat:
> Konsumenter som benytter individnummeret til å utlede korrekt fødselsår
for D-nummerpersoner må endre sine systemløsninger. Endringen vil
produksjonssettes medio august 2021 og det vil kunne tildeles
D-numre med nye nummerserier umiddelbart.
Copy link

@jorn-ola-birkeland jorn-ola-birkeland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jeg har bare sammenliknet spec'en med koden for å utlede århundre, og den biten ser riktig ut.

@eivinhb
Copy link
Member Author

eivinhb commented Jan 15, 2022

@jorn-ola-birkeland Det er jeg mest usikker på er egentlig min forståelse av at den ny oppdelingen av individnummerne, som før var like for D-nummer og Personnummer, og at de nå blir ulike. Jeg tror det blir riktig, men skulle gjerne visst.

@jorn-ola-birkeland
Copy link

Skjønner. Din tolkning virker rimelig, og at individnummer for personnumre er uendret, og endringen kun gjelder for D-nummer. Men jeg VET ikke, nei

@hcf
Copy link

hcf commented Jan 17, 2022

Spec -> kode ser bra ut fra mitt ståsted også, og minner meg på at vi må oppdatere vår egen kode...

Men merker at jeg blir litt forstyrret av manglende formattering, en autoformatering hadde kanskje vært på sin plass?

@eivinhb
Copy link
Member Author

eivinhb commented Jan 17, 2022

Men merker at jeg blir litt forstyrret av manglende formattering

Jeg er enig i at det ikke ser så allright ut på github, men idea er happy altså. Det er autoformatert, men det er litt tabs vs. spaces ute og går her. Jeg kan se om jeg kan få det over på tabs, som er det vanlige i NoCommons.

Autoformatert Fodselsnummer.java slik at den er i henhold.
@eivinhb eivinhb marked this pull request as ready for review January 17, 2022 08:27
.editorconfig Show resolved Hide resolved
@kvangaball
Copy link
Contributor

Snublet over denne og synes nå ihvertfall at det er supert å få inn d-nummerstøtte. Og for det det er verdt, så ser endringene bra og riktige og fine ut 😄

@eivinhb eivinhb merged commit a303c58 into master Feb 16, 2022
@eivinhb eivinhb deleted the dnummerutvidelse branch February 16, 2022 14:52
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.

5 participants