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

Nav 22888 småting tilbakekreving #2022

Merged
merged 10 commits into from
Jan 8, 2025
Merged

Conversation

amiraliw
Copy link
Contributor

@amiraliw amiraliw commented Jan 3, 2025

No description provided.

@amiraliw amiraliw requested a review from a team as a code owner January 3, 2025 10:11
@ViktorGSolberg
Copy link
Contributor

ViktorGSolberg commented Jan 3, 2025

Gi gjerne en forklaring av problemet PRen forsøker å løse samt lenke til det relevante Favro-kortet hvor oppgaven er spesifisert. Det gjør livet lettere for oss som skal se gjennom PRen 😊

@amiraliw
Copy link
Contributor Author

amiraliw commented Jan 3, 2025

PRen forsøker å løse:
Når man har gjort en endring i vilkår/foreldelse/fakta så bør knappen hete "Bekreft og fortsett", og ikke "Neste". Når man IKKE har gjort endringer kan den godt hete "Neste".
Favro link:
https://favro.com/organization/98c34fb974ce445eac854de0/86f241541982404a7491d82d?onShow=mycards&card=NAV-22888

Løsningen er da slik at hvis det skjer en endring i siden så navnet til kanppen endres fra "Neste" til "Bekreft og fortsett".
I Vilårsvurdering siden er det slik at Neste-knappen endres hvis Bekreft-knappen som er i perioden(e), er klikket.
Bekreft-knappen i perioden(e) er Disable når man åpner en periode. Den blir Enable hvis siden endres.
Alle endringene i perioden(e) blir lagret når man trykker på "Bekreft og fortsett" knappen i Vilårsvurdering siden.
Legg merke til at hvis man først endrer siden og endrer den igjen tilbake til den som var, før man trykker på noe knapp, er knappen fortsatt "Bekreft og fortsett" for at siden er egentlig endret (selv om den ble endret tilbake til den var).

Copy link
Contributor

@ViktorGSolberg ViktorGSolberg left a comment

Choose a reason for hiding this comment

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

Jeg testet dette lokalt og det ser ut til å funke bra 😄👍
Jeg synes forsåvidt dette er en god løsning, men jeg ser at i BehandlingContext så eskporterer vi en variabel harUlagredeData. Kunne man bare brukt denne til å utlede knappeteksten?

Da slipper man å ha to forskjellige states som sjekker det samme (harUlagredeData og erBehandlingEndret) i FaktaSkjema og de andre skjemaene

const { skjema, onBekreft } = useVilkårsvurderingPeriodeSkjema(
(oppdatertPeriode: VilkårsvurderingPeriodeSkjemaData) => {
oppdaterPeriode(oppdatertPeriode);
behandlingEndret(true);
Copy link
Contributor

Choose a reason for hiding this comment

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

Brukes denne til noe nå?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nei den skal fjernes

@@ -105,6 +105,7 @@ const VilkårsvurderingPerioder: React.FC<IProps> = ({
} = useFeilutbetalingVilkårsvurdering();

const tidslinjeRader = genererRader(perioder, valgtPeriode, erTotalbeløpUnder4Rettsgebyr);
const [erBehandlingEndret, settErBehandlingEndret] = React.useState<boolean>(false);
Copy link
Contributor

Choose a reason for hiding this comment

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

Denne kan antakeligvis byttes ut med harUlagredeData og settIkkePersistertKoponent etterhvert :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

bytta ut med harUlagredeData :)

amiraliw and others added 3 commits January 7, 2025 09:25
…t BehandlingProvider for å kunne teste navigering og enabled/disable funksjonalitet av knapper
…jema.tsx og brukte harUlagredeData istedenfor erBehandlingEndret i VilkårsvurderingPerioder.tsx. tilpasset også testene med disse endringer
Copy link
Contributor

@ViktorGSolberg ViktorGSolberg left a comment

Choose a reason for hiding this comment

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

Ser veldig bra ut 😊 Dette er ikke den enkleste appen å sette seg inn i så kudos til deg for at du gjør det! 👏

@amiraliw amiraliw merged commit 5847871 into main Jan 8, 2025
3 checks passed
@amiraliw amiraliw deleted the NAV-22888-Småting-tilbakekreving branch January 8, 2025 14:31
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