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

Hooks laddar inte om data om parametrar ändras #579

Open
whyer opened this issue Dec 2, 2021 · 1 comment
Open

Hooks laddar inte om data om parametrar ändras #579

whyer opened this issue Dec 2, 2021 · 1 comment
Labels
bugg Något är trasigt

Comments

@whyer
Copy link
Contributor

whyer commented Dec 2, 2021

Hittat en bugg i api-hooks som jag inte riktigt vet hur den ska fixas.
Om parametrarna till tex useTimetable ändras så ändras inte datan vi får tillbaka.
Hooks 'upptäcker' inte att parametrarna i tex child nedan ändras och ställer aldrig om frågan till api:t.

const { data: weekLessons } = useTimetable( child, week, year, LanguageService.getLanguageCode() )

Jag märkte att när man kör så syns inte schema-tiderna på framsidan ibland. Orsaken är att vi gör två anrop mot etjanst och skola24 för att sen merga ihop datat.
Det betyder att child-objektet ändras efter en stund när skola24 är laddat. Men om vi skickat iväg en anrop till useTimetable och useTimetable fått svar från api:t innan skola24 berikat child-objektet så får vi tillbaka en tom array. Och sen får man alltid tillbaka en tom array för den hooken (för den vyn).

Som jag ser det finns det inget i hooks som triggar om en ny load() om nyckeln (key) ändras.

@kajetan-kazimierczak kajetan-kazimierczak added the bugg Något är trasigt label Dec 2, 2021
@viktorlarsson
Copy link
Contributor

Har vi någon useEffect på parametrarna?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugg Något är trasigt
Projects
None yet
Development

No branches or pull requests

3 participants