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

feat: analyticspakker og bruk av dem i portalen #3300

Merged
merged 20 commits into from
Dec 5, 2022
Merged

Conversation

wkillerud
Copy link
Contributor

@wkillerud wkillerud commented Dec 1, 2022

Mål:

  • Få analytics i portalen.
  • Hindre tette koblinger til en enkeltleverandør i kode.
  • Unngå at alle finner opp hjulet mellomlaget på nytt.
  • Del et eksempel på en backend til mellomlaget som det er kjent at flere vil kunne trenge.

Det viktigste å ta stilling til er IMO:

  • Gir det mening å ha analyticspakker her i Jøkul (jøkulytics 🥁), eller skulle vi ha flyttet noe til et annet repo?
  • Ser du noen umiddelbare mangler i det delte APIet?

Mest relevante filer (packages/analytics/ fjernet i 1bd6d06 om du vil se diff):

  • packages/analytics**/README.md
  • packages/analytics/src/analytics.ts
  • packages/analytics/src/types.ts
  • packages/analytics-mixpanel/src/mixpanel.ts
  • portal/src/layout/portal-footer/Cookies.tsx
  • portal/src/layout/Layout.tsx

🎯 Sjekkliste

Hvis du trenger å styre UI basert på samtykker kan du bruke consents som returneres av
useCookieConsent.
Et delt mellomlag som kan utvides med flere backends. I første
omgang en backend for Mixpanel.
Unngå at en fremtidig multi-backend drar inn alt av dependencies for alle brukere
@wkillerud wkillerud added the ✨ enhancement Forslag til nye funksjoner og endringer label Dec 1, 2022
@wkillerud wkillerud self-assigned this Dec 1, 2022
@fremtind-bot
Copy link
Collaborator

fremtind-bot commented Dec 1, 2022

Forhåndsvisning: https://jokul.fremtind.no/preview/portal/analytics/
🔍 Commit: 971fdf2

Forhåndsvisningen blir tilgjengelig innen et par minutter. Den fjernes automatisk når pull requesten lukkes.

fremtind-bot added a commit that referenced this pull request Dec 1, 2022
kennidenni
kennidenni previously approved these changes Dec 2, 2022
Copy link
Contributor

@kennidenni kennidenni left a comment

Choose a reason for hiding this comment

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

Har ikke brukt mixpanel selv, men ser ganske bra ut

Copy link
Contributor

@piofinn piofinn left a comment

Choose a reason for hiding this comment

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

Synes dette ser veldig bra ut! 🎉

Jeg tror kanskje jeg heller mot at dette bør leve for seg selv, eller har et annet prefiks enn jkl. For meg føles dette i hvert fall ikke som en del av designsystemet, men heller som et generelt verktøy for frontendutvikling. Ikke at jeg vet hva det skulle hete eller hvor det skulle bodd, da 😅

portal/src/layout/portal-footer/Cookies.tsx Outdated Show resolved Hide resolved
portal/src/layout/portal-footer/PortalFooter.tsx Outdated Show resolved Hide resolved
@wkillerud
Copy link
Contributor Author

Jeg tror kanskje jeg heller mot at dette bør leve for seg selv, eller har et annet prefiks enn jkl. For meg føles dette i hvert fall ikke som en del av designsystemet, men heller som et generelt verktøy for frontendutvikling. Ikke at jeg vet hva det skulle hete eller hvor det skulle bodd, da 😅

Vi kan spinne opp et nytt public monorepo for delte ting som ikke er Jøkul, eller bare for analytics. Blir en oppstartskostnad, men da kan vi hindre at -utils-ting sklir ut enda mer enn det har gjort allerede. Vi kunne kanskje til og med flyttet ut et par ting:

  • constants-util
  • Kanskje formatters-util og validators-util? De er i grenseland.

Hvis vi er i gang med rydding kunne vi sikkert med fordel også delt actions/ opp i hvert sitt repo, så de er mulig å gjenbruke i andre repos sine workflows.

Vi får legge en plan til uka @piofinn, men da høres det ut som vi drøyer litt å lage pakker av analytics-koden inntil videre.

fremtind-bot added a commit that referenced this pull request Dec 5, 2022
den er ikke secret, men det er ryddigere med tanke på vedlikehold
fremtind-bot added a commit that referenced this pull request Dec 5, 2022
fremtind-bot added a commit that referenced this pull request Dec 5, 2022
fremtind-bot added a commit that referenced this pull request Dec 5, 2022
Uansett ikke så nøye, siden vi skiller på prosjektnivå mellom prod
og preview.
fremtind-bot added a commit that referenced this pull request Dec 5, 2022
@wkillerud wkillerud linked an issue Dec 5, 2022 that may be closed by this pull request
3 tasks
Copy link
Contributor

@piofinn piofinn left a comment

Choose a reason for hiding this comment

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

LGTM!

@wkillerud wkillerud merged commit 388dca2 into main Dec 5, 2022
@wkillerud wkillerud deleted the portal/analytics branch December 5, 2022 10:07
github-actions bot pushed a commit that referenced this pull request Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement Forslag til nye funksjoner og endringer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vi trenger noe besøkstatistikk i portalen
4 participants