Utvid tjeneste som gir inntektsmelding til fpoversikt #6790
Conversation
# Conflicts: # web/src/test/java/no/nav/foreldrepenger/web/app/tjenester/fpoversikt/InntektsmeldingDtoTjenesteTest.java
...idsforhold/src/main/java/no/nav/foreldrepenger/domene/iay/modell/InntektsmeldingBuilder.java
Outdated
Show resolved
Hide resolved
| @@ -187,7 +187,7 @@ private Optional<String> finnDokumentId(JournalpostId journalpostId, List<ArkivJ | |||
| .map(d -> d.getHovedDokument().getDokumentId()); | |||
| } | |||
|
|
|||
| private Optional<KontaktinformasjonIM> trekkUtKontaktInfo(MottattDokument mottattIM) { | |||
| public static Optional<KontaktinformasjonIM> trekkUtKontaktInfo(MottattDokument mottattIM) { | |||
There was a problem hiding this comment.
Trengte tilgang til denne andre steder. Usikke rpå om dette var beste løsning
There was a problem hiding this comment.
Kunne kanskje heller laget en statisk klasse som kan tas inn begge steder? Så slipper klasser som egentlig bare vil hente ut KontaktInfo å dra inn hele ArbeidOgInntektsmeldingDtoTjeneste
| import no.nav.foreldrepenger.domene.typer.JournalpostId; | ||
|
|
||
| @JsonInclude(JsonInclude.Include.NON_NULL) | ||
| record FpOversiktInntektsmeldingDto(Boolean erAktiv, BigDecimal stillingsprosent, |
There was a problem hiding this comment.
Vil noen sjekke om deres editor formattering gjør denne pen? Usikker om min funker som intended
| @@ -84,6 +84,24 @@ fpsak.it.ps.scopes=testscope | |||
| spokelse.grunnlag.url=http://localhost:8060/rest/spokelse/grunnlag | |||
| spokelse.grunnlag.scopes=testscope | |||
|
|
|||
| #TOKEN_X_WELL_KNOWN_URL=http://localhost:8060/rest/tokenx/.well-known/oauth-authorization-server | |||
There was a problem hiding this comment.
oops, fikk denne commited ved et uhell. Kan reverte. Men husker du da den problematikken oppstod @pekern ? Trenger jeg fremdeles å ha en annerledes lokal konfig her? Har bare hatt den liggende uncommited siden da
There was a problem hiding this comment.
Nei, du skal ikke trenge noe lokal konfig lenger, appplication-vtp.properties på master skal være rett. Kan nok bare ta vekk denne :)
|
|
||
| List<FpOversiktInntektsmeldingDto.NaturalYtelse> bortfalteNaturalytelser = new ArrayList<>(); | ||
|
|
||
| gruppertPåType.forEach((key, value) -> { |
There was a problem hiding this comment.
Denne er hjulpet av chatgpt til å oversette fra JS. Tar innspill til om den kan prettifies med ekte java
There was a problem hiding this comment.
Ikke utenkelig at denne kan løses penere med tidsserier, vi har noen eksempler på det rundt i løsningen. Søk på LocalDateTimeline / LocalDateSegment i fpsak. Men hvis du synes det virker tungvint tenker jeg dette er greit til å begynne med.
| @ExtendWith(JpaExtension.class) | ||
| @ExtendWith(MockitoExtension.class) |
There was a problem hiding this comment.
Trenger du både JpaExtension og MockitoExtension? Bør det ikke holde med mockito?
There was a problem hiding this comment.
Tror jeg trenger JpaExtension for EntityManager bruk
|
|
||
| List<FpOversiktInntektsmeldingDto.NaturalYtelse> bortfalteNaturalytelser = new ArrayList<>(); | ||
|
|
||
| gruppertPåType.forEach((key, value) -> { |
There was a problem hiding this comment.
Ikke utenkelig at denne kan løses penere med tidsserier, vi har noen eksempler på det rundt i løsningen. Søk på LocalDateTimeline / LocalDateSegment i fpsak. Men hvis du synes det virker tungvint tenker jeg dette er greit til å begynne med.
No description provided.