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

PE-IY-05-200 og -201 Godskriving omsorgspoeng #1043

Open
wants to merge 33 commits into
base: main
Choose a base branch
from

Conversation

madsop-nav
Copy link
Contributor

Delt i tre forskjellige malar, som det eigentleg var under panseret også i Exstream

Korresponderande PR i PEN: https://github.com/navikt/pensjon-pen/pull/14151

@madsop-nav madsop-nav requested a review from a team as a code owner November 13, 2024 08:32
import no.nav.pensjon.brev.api.model.maler.RedigerbarBrevdata
import no.nav.pensjon.brev.api.model.maler.legacy.OmsorgLegacyData

data class OmsorgLegacyDataDto(
Copy link
Contributor

Choose a reason for hiding this comment

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

Hvert brev bør ha sin egen data klasse så det blir lettere å utvide datagrunnlaget pr brev i fremtiden.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tanken med denne er å lage ein wrapper for OmsorgLegacyData, som kun bruker den, ut frå hypotesa om at det vil gjelde for alle legacy-omsorgsbreva vi konverterer over, og at så lengje dei framleis bruker den klassa og ikkje dei relevante felta direkte vil det uansett kun vera denne modellen, sånn at vi da lager separate dataklassar når vi tar dei ut av legacy.

brevtype = LetterMetadata.Brevtype.INFORMASJONSBREV
)
) {
//IF( PE_pebrevkode = "PE_IY_05_201" AND (FF_GetArrayElement_Integer(PE_Grunnlag_OmsorgGodskrGrunnlagListe_OmsorgGodskrGrunnlagAr,1) > 1991 OR Year(PE_PersonSak_Fodselsdato) < 1948 OR Year(PE_PersonSak_Fodselsdato) > 1953) )THEN INCLUDE ENDIF
Copy link
Contributor

Choose a reason for hiding this comment

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

Ser du har ett par kommentarer med exstream kode, men ingen showIf. Skal dette styres ved at noen bestiller riktig brev bare istedenfor? Da kan du evt slette logikk-komentarene om det stemmer.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ja, det er tanken - det var mykje logikk her for å skilje tekst til desse tre breva.

Men lurer på om det eigentleg burde vore logikk i PEN for å seie kva av breva som skal vera tilgjengeleg når, det er jo styrt ut frå data vi har (fødselsdato samt når omsorgs-året var) 🤔

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.

3 participants