Skip to content
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

Diverse vtao-fikser #1295

Merged
merged 7 commits into from
Feb 27, 2025
Merged

Diverse vtao-fikser #1295

merged 7 commits into from
Feb 27, 2025

Conversation

Oddsor
Copy link
Contributor

@Oddsor Oddsor commented Feb 24, 2025

Mange små commits gjør en stor PR

@Oddsor Oddsor requested review from sindredl and MagnusRom February 24, 2025 19:19
@Oddsor Oddsor self-assigned this Feb 24, 2025
@Oddsor Oddsor force-pushed the vtao-fikser branch 2 times, most recently from 785f67e to bffc105 Compare February 27, 2025 08:53
Opprett ny "fortsett tiltak"-modal som erstatter
tidligere "oppfølging gjennomført".

En ny "feature" er at vi viser når siste oppfølging
ble gjennomført.
Teksten skal kun vises når forrige tilskuddsperiode er behandlet,
dvs når tilskuddsperioden er godkjent.
handterFeil-metoden håndterte ikke faktisk feil av typen
IkkeTilgangError, slik at man ble sittende med en "uventet feil".

Denne endringen gjør at feilmeldingskode kan brukes for 403-feil,
feks når man oppretter avtaler på deltaker man ikke har tilgang til.
@Oddsor Oddsor force-pushed the vtao-fikser branch 2 times, most recently from 4a63f07 to ecf2772 Compare February 27, 2025 13:04
I stedet for å se på datoen når oppfølging skal utføres, og kalkulere om
den er etter dagens dato, kan vi sjekke om feltet oppfolgingVarselSendt
er ulik `null` for å oppnå det samme.

Feltet blir satt til `null` når oppfølging er utført, før det igjen settes ved
neste oppfølging.
@Oddsor Oddsor requested a review from sindredl February 27, 2025 13:13
@Oddsor Oddsor merged commit 70d3287 into master Feb 27, 2025
1 check passed
@Oddsor Oddsor deleted the vtao-fikser branch February 27, 2025 14:23
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.

3 participants