-
Notifications
You must be signed in to change notification settings - Fork 1
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
Ut av sanity #1491
Conversation
…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.
…, da den hører mer hjemme der.
There was a problem hiding this 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)
There was a problem hiding this 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.
Har fjernet endringene jeg gjorde med nynorsk, fjernet ubrukt typer, bruker localetekst på alerten. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ser bra ut 👏
#1495) Byttet til ny Aksel DatePicker for mindre sikkerhetsissues og færre gamle avhengigheter.
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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