-
Notifications
You must be signed in to change notification settings - Fork 2
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
Innsatsbehovsendring på en avtale må lagres ved godkjenning #1261
Conversation
Co-authored-by: Mickel Hoang <mickel.hoang@nav.no>
avtale.forlengAvtale(sluttDato, getIdentifikator()); | ||
} | ||
|
||
protected void oppdatereEnheterEtterForespørsel(Avtale avtale) { | ||
final PdlRespons persondata = this.hentPersonDataForOpprettelseAvAvtale(avtale); | ||
this.sjekkOgHentOppfølgingStatus(avtale, veilarboppfolgingService); | ||
this.oppdaterOppfølgningsStatusForAvtale(avtale); // TODO: Fjern denne slik at veileder kan oppdatere enhet uten å oppdatere oppfolgingstatus |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -319,8 +319,9 @@ private PdlRespons hentPersondata(Fnr deltakerFnr) { | |||
return persondata; | |||
} | |||
|
|||
public void sjekkOgHentOppfølgingStatus(Avtale avtale, VeilarboppfolgingService veilarboppfolgingService) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
metoden heter hent men returnerer ikke noe -> endret derfor metodenavn til oppdatere
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Den sjekker og om oppfølgingen er gyldig, så kanskje sjekkOgOppdaterOppfølgningsstatus
? 🤔
…i Veileder.java som gjenspeiler hva metoden gjør
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Det eneste jeg hang meg litt opp i er at vi ved godkjenning også lagrer oppfølgingsenhet, men jeg tror ikke det gjør noe.. 👍
No description provided.