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

Ut av sanity #1491

Merged
merged 16 commits into from
Nov 21, 2023
Merged

Ut av sanity #1491

merged 16 commits into from
Nov 21, 2023

Conversation

gunnsteingarmo
Copy link
Contributor

Hvorfor er denne endringen nødvendig? ✨

Har tatt tekst ut fra Sanity og lagret det i koden. Teksten brukes av komponenter for hver av søknadene. Har fjernet Sanity som avhengighet i prosjektet.

Favro: https://favro.com/organization/98c34fb974ce445eac854de0/a64c6aad9b0d61ef6c0290bd?card=NAV-16300

…ter som skal gjenbrukes, tatt tekst inn i filer, refaktorert kode blant annet typet mer.
…ode mer lettlest og gjenbrukbar. Teksten som skal vises på barnetilsyn er nå tatt over i kode og brukes i egen komponent for informasjon om barnetilsyn.
… i stedet for Sanity. Tatt kode ut i komponenter for å enklere gjenbruke kode. Fjernet Sanity fra prosjektet.
Copy link
Collaborator

@ma10s ma10s left a comment

Choose a reason for hiding this comment

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

m3pb011r.apicdn.sanity.io - denne kan kanskje fjernes i samme pr? (csp.ts)

src/utils/typer.ts Outdated Show resolved Hide resolved
Copy link
Collaborator

@ma10s ma10s left a comment

Choose a reason for hiding this comment

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

Ser bra ut - har testet ganske grundig i preprod og ikke funnet noe som mangler, eller er feil. Veldig bra. (Men kanskje fint om en som kan frontend ser på koden ;D også )

Noen små spørsmål som du kanskje kan se på før du merger?

…tOversettelse bruker localetekst slik aom andre alerts.
@gunnsteingarmo
Copy link
Contributor Author

Ser bra ut - har testet ganske grundig i preprod og ikke funnet noe som mangler, eller er feil. Veldig bra. (Men kanskje fint om en som kan frontend ser på koden ;D også )

Noen små spørsmål som du kanskje kan se på før du merger?

Har fjernet endringene jeg gjorde med nynorsk, fjernet ubrukt typer, bruker localetekst på alerten.

Copy link
Contributor

@ViktorGSolberg ViktorGSolberg left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Contributor

@charliemidtlyng charliemidtlyng left a comment

Choose a reason for hiding this comment

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

Ser bra ut 👏

@gunnsteingarmo gunnsteingarmo merged commit 5b2e317 into master Nov 21, 2023
3 checks passed
@gunnsteingarmo gunnsteingarmo deleted the ut-av-sanity branch November 21, 2023 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants