Skip to content

Conversation

@remhol
Copy link
Collaborator

@remhol remhol commented Oct 30, 2025

  • 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

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.

* 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
@remhol remhol requested a review from a team as a code owner October 30, 2025 13:34
@remhol remhol requested review from a team, AnjaAalerud, Copilot, pekern and tor-nav October 30, 2025 13:34
Copy link
Contributor

Copilot AI left a 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 RefusjonTilVurderingKravForSent for late refund claims
  • Creates new component VurderRefusjonKravForSentRad to 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.

@remhol
Copy link
Collaborator Author

remhol commented Oct 30, 2025

Har ikke satt igang med testene enda, men tenkte bare å publisere pr-en nå slik at man får sett på selve koden

"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 {}}",
Copy link
Collaborator

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

Copy link
Collaborator Author

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? 😄

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 3, 2025

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Knip report

Unlisted dependencies (1)

Name Location Severity
@navikt/ds-css .storybook/preview.tsx:7:31 error

"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 {}}",

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" ?

@remhol
Copy link
Collaborator Author

remhol commented Dec 11, 2025

Lukker denne i påvente av avklaringer som trengs for videre utvikling

@remhol remhol closed this Dec 11, 2025
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.

5 participants