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

Legg inn sanity-tekster på forsiden #1139

Merged
merged 7 commits into from
Mar 18, 2024
Merged

Legg inn sanity-tekster på forsiden #1139

merged 7 commits into from
Mar 18, 2024

Conversation

idaame
Copy link
Contributor

@idaame idaame commented Mar 14, 2024

💰 Hva forsøker du å løse i denne PR'en

Skriv 1 eller 2 setninger om hvilken funksjonell endring som blir implementert.
https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-18565

Flytter over resten av tekstene som ligger bak KOMBINER_SØKNAD-toggle til Sanity.

🔎️ Er det noe spesielt du ønsker å fremheve?

Er det noe du er bekymret eller usikker på? Beskriv det gjerne her.
Nei. Kommer til å teste i preprod før merge, med både toggle av og på.

✅ Checklist

Har du husket alle punktene i listen?

  • Jeg har testet mine endringer i henhold til akseptansekriteriene/skissene 🕵️
  • Jeg har testet endringene mine i mobilstørrelse, zoom 200%, skalerer riktig med endret tekststørrelse i browser 📱
  • Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇
  • Jeg har fikset en bug, og skrevet regresjonstest for denne
  • Jeg har endret søknadskontrakten og modellversjon i Miljø.ts

Jeg har ikke skrevet tester fordi:
Tester manuelt at ting funker som det skal

🤷‍♀ ️Hvor er det lurt å starte?

F.eks. commit for commit, alt i ett?
Alt i ett er sikkert like greit, har vært litt frem og tilbake

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei, har jobbet sammen med @daphneleebeek så trenger ikke gå gjennom muntlig hvis det ikke er ønske

👀 Screen shots

Har det visuelle endret seg? Legg til før- og etterbilder!
image

@idaame idaame marked this pull request as ready for review March 15, 2024 12:28
@idaame idaame merged commit d86c677 into main Mar 18, 2024
8 checks passed
@idaame idaame deleted the sanity/tekster-forside branch March 18, 2024 08:20
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.

2 participants