IS-2828: Add amplitude logging for sok #554
Conversation
| useEffect(() => { | ||
| if (isSuccess) { | ||
| logSokPersonResults(searchResults.length); | ||
| } | ||
| }, [isSuccess, searchResults]); |
There was a problem hiding this comment.
Kunne man løst det uten en useEffect? Men er kanskje riktig, siden det er en sideeffekt.
There was a problem hiding this comment.
mutate har vel en sånn onSuccess greie🤔 Går det an å logge det der? 😁 Se TildeltVeileder for eksempel
There was a problem hiding this comment.
Ja true, det er jo kanskje bedre 👍🏼
There was a problem hiding this comment.
Ikke noe galt med useEffect her tror jeg, men synes å legge det i onSuccess-callbacken på mutate leser bedre 👍 Begge loggingene burde kanskje ligge i en sånn callback? Evt putte logSokPersonEvent i onSettled hvis vi vil logge den nå kallet har fullført uansett error/success.
There was a problem hiding this comment.
Det var smart, funka bra 👍🏼
| @@ -11,6 +11,7 @@ export enum EventType { | |||
| OptionSelected = 'alternativ valgt', | |||
| ButtonClick = 'knapp trykket', | |||
| SortingColumn = 'kolonne sortert på', | |||
| AmountDisplayed = 'antall vist', | |||
There was a problem hiding this comment.
Tar gjerne tilbakemeldinger på navn her og data-objektet i typen. Sånn jeg forstår det skal dette helst være handlinger, men antall resultater er jo mer en handling av systemet og ikke brukeren 🤔
There was a problem hiding this comment.
Tenker det er gudd jeg 😊 Evt. 'antall resultater'. Men fint slik som det er også
eirikdahlen
force-pushed
the
IS-2828-amplitude-sok
branch
from
ed8172b
to
3663b71
Compare
Hva har blitt lagt til✨🌈
Håper dette skal fikse:
Og så har vi allerede målinger på klikk på person/lenke, som vi kan knytte opp til søk-klikket i en funnel.