-
Notifications
You must be signed in to change notification settings - Fork 3
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
EY-4555: Endrer adresse for brev ved bosatt utland #965
base: main
Are you sure you want to change the base?
Conversation
showIf(utland) { | ||
paragraph { |
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.
Dette er nesten likt det som finnes i common.Postadresse bortsett fra avstand mellom linjene. Må sjekke opp om det kan gjøres likt for å unngå duplisering her.
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.
Hvis jeg ikke husker helt feil, så er var det noe problem med å få ting på riktig linje i redigerbare brev og at det derfor er gjort slik. Mulig du burde spørre @EspenSchei om det, da jeg tror han satt med det.
) : FerdigstillingBrevDTO | ||
val bosattUtland: Boolean = false, | ||
) : FerdigstillingBrevDTO { | ||
val informasjonOmOmstillingsstoenadData = InformasjonOmOmstillingsstoenadData(tidligereFamiliepleier, bosattUtland) |
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.
Eneste måten jeg ser for å "bundle" to parametere sammen og så sende dem til et vedlegg som brukes på tvers av ulike brev/dto'er. Alternativet som er gjort en del andre steder hadde vært å ha mange ulike includes, men det ser jeg som en enda kjipere løsning her
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.
Dette er en mye bedre løsning!
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.
Looks good!
Avhengig av endringer i brev-api også for å sende over
bosattUtland
.