-
Notifications
You must be signed in to change notification settings - Fork 2
TFP-6405: Legger til AP for refusjonsfrist i fakta fordeling #3924
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
Conversation
* Legges til over vurdering av endret refusjon * Beholder ap for refusjon i fakta beregning for saker fra før vi skrur på toggle i fp-kalkulus
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.
Pull Request Overview
This PR adds support for handling late refund claims (refusjonskrav for sent) in the calculation basis distribution. The main change introduces a new type of refund claim that can be validated separately from regular refund claims that overlap with previous payments.
- Adds new type
RefusjonTilVurderingKravForSentfor late refund claims - Creates new component
VurderRefusjonKravForSentRadto display and validate late refund claims - Updates form structure to separate regular refund fields from late claim fields
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/types/src/BeregningsgrunnlagFordeling.ts | Adds type definition for late refund claims and adds list to RefusjonTilVurdering |
| packages/types/index.ts | Exports the new RefusjonTilVurderingKravForSent type |
| packages/fakta-fordel-beregningsgrunnlag/src/types/interface/VurderRefusjonBeregningsgrunnlagAP.ts | Adds transformed values type for late claims and makes fastsatteAndeler optional |
| packages/fakta-fordel-beregningsgrunnlag/src/types/FordelBeregningsgrunnlagPanelValues.ts | Restructures form values to separate refusjon and refusjonskrav fields |
| packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderRefusjonKravForSentRad.tsx | New component for rendering and handling late refund claim rows |
| packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderEndringRefusjonRad.tsx | Updates field names to nest under 'refusjon' namespace |
| packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderEndringRefusjonForm.tsx | Updates buildInitialValues and transformFieldValues to handle both andeler and refusjonskravForSentListe |
| packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderEndringRefusjonField.tsx | Conditionally renders both late claims and regular refund sections, updates help text |
| packages/fakta-fordel-beregningsgrunnlag/i18n/nb_NO.json | Adds Norwegian translations for late refund claim UI |
Comments suppressed due to low confidence (1)
packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderRefusjonKravForSentRad.tsx:1
- Remove the '.tsx' file extension from the import statement. TypeScript module imports should not include file extensions.
import { useFormContext } from 'react-hook-form';
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderEndringRefusjonField.tsx
Outdated
Show resolved
Hide resolved
packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderEndringRefusjonForm.tsx
Outdated
Show resolved
Hide resolved
packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderEndringRefusjonForm.tsx
Outdated
Show resolved
Hide resolved
...ges/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderRefusjonKravForSentRad.tsx
Outdated
Show resolved
Hide resolved
|
Har ikke satt igang med testene enda, men tenkte bare å publisere pr-en nå slik at man får sett på selve koden |
packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderEndringRefusjonForm.tsx
Outdated
Show resolved
Hide resolved
packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderEndringRefusjonForm.tsx
Outdated
Show resolved
Hide resolved
...ges/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderRefusjonKravForSentRad.tsx
Outdated
Show resolved
Hide resolved
packages/fakta-fordel-beregningsgrunnlag/src/components/refusjon/VurderEndringRefusjonField.tsx
Outdated
Show resolved
Hide resolved
…dring i refusjon * Fikser tester som feilet
| "BeregningInfoPanel.RefusjonBG.TidligereRefusjon": "<b>{ag}</b> krever refusjon fra og med {dato}. Det er tidligere innvilget et lavere refusjonsbeløp", | ||
| "BeregningInfoPanel.RefusjonBG.RefusjonFra": "Refusjonsbeløpet skal gjelde fra og med", | ||
| "BeregningInfoPanel.RefusjonBG.Aksjonspunkt": "Nytt refusjonskrav overlapper tidligere utbetalinger. Sett endringsdato for ny refusjon.", | ||
| "BeregningInfoPanel.RefusjonBG.Aksjonspunkt": "<b>Vurder refusjonskrav</b> {br} {harRefusjonskravForSent, select, true {Refusjonskrav er mottatt etter fristen. Vurder om krav skal tas med i beregningen.} other {}} {skalHaBr, select, true {{br}} other {}} {harAndeler, select, true {Nytt refusjonskrav overlapper tidligere utbetalinger. Sett endringsdato for ny refusjon.} other {}}", |
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 Tor foreslo å løse det slik, så det er mulig det bare er et meg problem, men jeg syns denne er ganske tung å lese. hadde det kanskje vært bedre å bryte den opp i flere setninger? Så blir det mindre logikk i .json fila og mer i .tsx fila der jeg tenker det hører hjemme
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.
Litt sånn som jeg hadde det i utgangspunktet eller noe annet igjen? 😄
|
Knip reportUnlisted dependencies (1)
|
| "BeregningInfoPanel.RefusjonBG.TidligereRefusjon": "<b>{ag}</b> krever refusjon fra og med {dato}. Det er tidligere innvilget et lavere refusjonsbeløp", | ||
| "BeregningInfoPanel.RefusjonBG.RefusjonFra": "Refusjonsbeløpet skal gjelde fra og med", | ||
| "BeregningInfoPanel.RefusjonBG.Aksjonspunkt": "Nytt refusjonskrav overlapper tidligere utbetalinger. Sett endringsdato for ny refusjon.", | ||
| "BeregningInfoPanel.RefusjonBG.Aksjonspunkt": "<b>Vurder refusjonskrav</b> {br} {harRefusjonskravForSent, select, true {Refusjonskrav er mottatt etter fristen. Vurder om krav skal tas med i beregningen.} other {}} {skalHaBr, select, true {{br}} other {}} {harAndeler, select, true {Nytt refusjonskrav overlapper tidligere utbetalinger. Sett endringsdato for ny refusjon.} other {}}", |
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.
Kunne vi endret denne setningen: "Sett endringsdato for ny refusjon." til feks "Sett startdato for når nytt refusjonskrav skal gjelde fra eller "Sett startdato for nytt refusjonskrav" ?
|
Lukker denne i påvente av avklaringer som trengs for videre utvikling |



Merk:
Denne flyttingen er bare del 1. I del 2 skal vi redesigne hele refusjons-delen. Det vil si at denne koden kun er en midlertidig løsning mens vi avklarer design og veien videre for refusjon.