-
Notifications
You must be signed in to change notification settings - Fork 0
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
Hybridbrev: Opnar for å redigere (delar av) brev frå Brevbakeren #1840
Conversation
apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/brevbaker/BrevbakerKlient.kt
Outdated
Show resolved
Hide resolved
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.
Jeg har ikke grundig oversikt over hvilke problemer det var med den forrige approachen / hva den forrige approachen var, men jeg synes det virker som en grei løsning for å få inn en "strukturert" redigerbar tekst.
En ting som slår meg som potensielt uheldig er at denne flyten for brevene vil alltid vise et redigeringsvindu for saksbehandler? Jeg tror en flyt der redigeringsvindu / muligheter er unntaket er best for kvaliteten av det som systemet sender ut på sikt, men det er kanskje mer en faglig avklaring :p
apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/BlockTilSlateKonverterer.kt
Outdated
Show resolved
Hide resolved
apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/BlockTilSlateKonverterer.kt
Outdated
Show resolved
Hide resolved
private fun opprettInnhold(behandling: Behandling, prosessType: BrevProsessType): BrevInnhold { | ||
private fun manueltBrevData(brev: Brev) = ManueltBrevData(requireNotNull(db.hentBrevPayload(brev.id)).elements) | ||
|
||
private suspend fun opprettInnhold(behandling: Behandling, prosessType: BrevProsessType): BrevInnhold { | ||
val tittel = "Vedtak om ${behandling.vedtak.type.name.lowercase()}" | ||
|
||
val payload = when (prosessType) { |
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.
Hadde det gitt mening med en tredje prosesstype for brev? I denne og forrige when
er det basically en sjekk på om du er en spesiell versjon av automatisk i kraft av revurderingsårsak, men det kunne vært basert på f.eks. prosessType REDIGERBAR_AUTOMATISK
e.l.
Har ikke oversikt over de andre stedene det evt. må håndteres da :p
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.
Oh, det er ein fristande ide. Samtidig trur eg at vi etter kvart skal over på redigerbar automatisk for alle brev, så usikker på om det vil trengs på sikt 🤔
@@ -13,7 +13,7 @@ data class Slate( | |||
@JsonValue val elements: List<Element> = emptyList() | |||
) { | |||
fun flettInn(behandling: Behandling): Slate = when (behandling.revurderingsaarsak) { | |||
RevurderingAarsak.ADOPSJON -> erstatt(this, BrevDataMapper.fra(behandling).second) | |||
RevurderingAarsak.ADOPSJON -> erstatt(this, BrevDataMapper.brevData(behandling)) |
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.
Ser at omgjøring av farskap er litt med og litt ikke med i denne PR'en -- hadde vært lettere å følge med hvis det var alt eller ingenting for den revurderingsårsaken
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.
Godt poeng! Noterer det bak øyret til neste gong.
apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/brevbaker/BrevbakerKlient.kt
Show resolved
Hide resolved
Det er viktig for fag å kunne redigere alle brev, iallfall. Men liker eigentleg tanken om at du kjem rett til forhåndsvisning, og så kan du klikke deg over på redigering viss det trengs. Det går jo an å lodde stemninga hos Merethe og Liv Inger om det. |
…havlege byrja å bli omfattande og krevjande å henge med på, og dels for å førebu redigerbar del av brev
…te heller enn å etterlikne den i vår eigen kodeß
This reverts commit 2d0eeec.
…onBuilder-nivå, så må konfigurere der
Denne heng saman med navikt/pensjonsbrev#196
Tanken her er i grunn å innføre konseptet delmalar.
Alle opphørsmalane skal ha samme format og ein del av dei samme vedlegga, så her innfører eg ein generell opphørs-mal. Når brukaren i Gjenny opnar malen, får hen den spesifikke delen av malen som er tenkt å kunne redigerast (som i denne første runden er omgjøring av farskap-malen). Når hen har redigert ferdig og ferdigstilt, tar vi innhaldet frå det hen har skrive og sender det inn som blokk-innhald til opphørs-malen.