Legger til UX signals på kvitteringsside #1495
Conversation
| /* empty */ | ||
| } | ||
| }; | ||
| }, [ready]); |
There was a problem hiding this comment.
Burde consent være med i dependency array? Usikker på hvordan consent fungere 😅
There was a problem hiding this comment.
Consent skal gi tilbake enten et definert objekt med rettigheter om brukeren har tatt stilling til cookie banneret, eller undefined hvis brukeren ikke har tatt stilling så vidt jeg skjønner. Siden dette ikke er en verdi som vi kommer til å lytte på når endrer seg tror jeg ikke vi trenger å ta den med i dependency-arrayet. 😊
There was a problem hiding this comment.
Dette er første gang jeg tar i bruk consent-greiene fordi denne oppgaven hastet noe å få ferdig, skal også legge det på Amplitude i en annen oppgave. Så det kan hende måten vi sender med consent endrer seg når jeg får sett på det mer helhetlig. 😄
src/frontend/hooks/useUxSignals.ts
Outdated
| }, [ready]); | ||
| }; | ||
|
|
||
| export default useUxSignals; |
There was a problem hiding this comment.
Ingen stor sak, men foretrekker som regel "named exports" pga. refaktoreringsproblemet nevnt her https://basarat.gitbook.io/typescript/main-1/defaultisbad
There was a problem hiding this comment.
Enig, jeg fikser 🙌
💰 Hva forsøker du å løse i denne PR'en
Legger til UX signals på kvitteringssiden til barnetrygdsøknaden, samme som vi allerede har på KS-søknaden.
🔎️ Er det noe spesielt du ønsker å fremheve?
Ingenting spesielt
✅ Checklist
Har du husket alle punktene i listen?
Jeg har ikke skrevet tester fordi:
Ikke relevant
🤷♀ ️Hvor er det lurt å starte?
Alt i ett
💬 Ønsker du en muntlig gjennomgang?
👀 Screen shots
Ikke relevant