Skip to content

Conversation

johannbm
Copy link
Collaborator

@johannbm johannbm commented Oct 2, 2025

No description provided.

@johannbm johannbm requested a review from a team as a code owner October 2, 2025 11:02
Copy link

sonarqubecloud bot commented Oct 2, 2025

| VurdereAnnenYtelseForVedtakAp
| UttakAp
| BeregningAksjonspunktSubmitType
| Partial<BeregningAksjonspunktSubmitType>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Usikker om dette er beste løsning. Er litt rart å inkludere en submitType her som ikke har samme optionals som andre?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ja den er litt pussig denne typen, men dette er vel neppe beste måte å løysa på. Kanskje greit med ts-ignore inntil vidare?

saksnummer: string;
bekreftedeAksjonspunktDtoer: ({
'@type': string;
'@type': string | undefined; // TODO
Copy link
Collaborator

Choose a reason for hiding this comment

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

bør ha kommentar

saksnummer: string;
overstyrteAksjonspunktDtoer: ({
'@type': string;
'@type': string | undefined; // TODO
Copy link
Collaborator

Choose a reason for hiding this comment

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

kommentar

riktigYtelseBeløp: number;
};

type VedtakResultatType = 'INGEN_TILBAKEBETALING' | 'DELVIS_TILBAKEBETALING';
Copy link
Collaborator

Choose a reason for hiding this comment

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

denne kan byttast ut med VedtakResultatType i fp-types

export const usePanelDataContext = <AP_TYPE extends AksjonspunktType>() => {
// @ts-expect-error Johannes ser på denne - mismatch mellom type i ft-repo og generert type
const context = useContext<Props<AP_TYPE> | null>(PanelDataContext);
const context = useContext(PanelDataContext);
Copy link
Collaborator

Choose a reason for hiding this comment

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

dette er vel det same som ligg på master

| VurdereAnnenYtelseForVedtakAp
| UttakAp
| BeregningAksjonspunktSubmitType
| Partial<BeregningAksjonspunktSubmitType>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ja den er litt pussig denne typen, men dette er vel neppe beste måte å løysa på. Kanskje greit med ts-ignore inntil vidare?

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.

2 participants