Fix stale data when clicking back button to stateless app #2565
Conversation
| queryKey, | ||
| queryFn: url ? () => fetchFormData(url, options) : skipToken, | ||
| enabled: true, | ||
| gcTime: 0, |
There was a problem hiding this comment.
Skal ikke hindre merge, men bare noen tanker 😄
Kunne det vært nyttig å sette staleTime i stedet for gcTime for at senere fetches skal hente med stale-while-revalidate?
En annen tanke er også at isStateless-variabelen har noe å si hvor hva som returneres, og burde da kanskje være i queryKey-en? Typ [...queryKey, { isStateless }] eller noe sånn at man har forskjellig queryKey for stateful og stateless.
There was a problem hiding this comment.
Hvis vi har har stale-while-revalidate, vil vi ikke da vise gammel data til vi har fått ny data? Det vil vi ikke i det tilfellet denne buggen kommer fra ihvertfall siden de bruker denne dataen til å vise om man har startet på skjemaet eller ikke. Vi ville da vist siden med "start nytt skjema" først, og så "Fortsett på skjema" når dataen er lastet.
Litt usikker på hva vi vinner på å ha isStateless i queryKey? At vi trigger query når isStateless forandrer seg?
There was a problem hiding this comment.
Query keyen bør allerede være forskjellig når man er i stateless siden query keyen er basert på url-en den henter ifra. For stateful er det url til data-elementet, men for stateless er det en annen url som refererer til data-typen som brukes i stateless (siden det ikke finnes noe data element da)
|
Description
Disabling cache of formdata when in a stateless app.
Testing:
Test users:
d-uka-2;894xt
d-duell-2;kf84d
d-viser-1;dyryx
d-felt-1;rx2c2
d-glis-2;nqbsk
d-kutt-1;vsxr7
Related Issue(s)
Verification/QA
This is a very custom implementation from UDIR that we cant easily test.
kind/*label to this PR for proper release notes grouping