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

feat: legge til hentUtvidetOrganisasjon i EReg-klient #1358

Merged
merged 2 commits into from
Apr 12, 2024

Conversation

sirimykland
Copy link
Contributor

No description provided.

@sirimykland sirimykland requested a review from a team as a code owner April 11, 2024 10:39
Copy link
Collaborator

@jolarsen jolarsen left a comment

Choose a reason for hiding this comment

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

LGTM, men vi kan vurdere å merge UtvidetOrganisasjonEReg og OrganisasjonEReg
Data fra kilden er nesten likt (nr, type, orgdetaljer) så skilles de på navn+virksomhetDetalj vs driverVirksomheter. Det er bare spm om hvilke get'ere som tilbys.

Interface er ikke noe poeng siden Jackson liker klasse-spec. Eneste bruk kunne være at noe krav om

@sirimykland
Copy link
Contributor Author

LGTM, men vi kan vurdere å merge UtvidetOrganisasjonEReg og OrganisasjonEReg Data fra kilden er nesten likt (nr, type, orgdetaljer) så skilles de på navn+virksomhetDetalj vs driverVirksomheter. Det er bare spm om hvilke get'ere som tilbys.

Jeg tenker det i denne omgang hadde vært praktisk å fått releaset dette slik at jeg kan fortsette å jobbe på TFP-5750 i fp-risk. Jeg liker tanken om å jobbe litt mer med å forbedre og gjøre UtvidetOrganisasjonEReg og OrganisasjonEReg likere. Kanskje vi kan gjøre det i en seperat pr, så kan det komme med i neste release?

@jolarsen
Copy link
Collaborator

Merge i vei, så kan vi release felles i dag

@jolarsen jolarsen merged commit 8032cf0 into master Apr 12, 2024
2 of 3 checks passed
@jolarsen jolarsen deleted the feature/TFP-5750 branch April 12, 2024 08:35
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