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

Legger til sanity tekster i "Om barnet" steget #1439

Merged

Conversation

ole-kristian-rudjord
Copy link
Contributor

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

  • Legger til Sanity tekster i "Om barnet" steget.
  • Lager "...ForSanity" komponenter slik at det er mulig å vise felt med Sanity tekster.
  • Endrer tester og enkelte komponenter til å ta i bruk data-testid. Dette gjøres fordi tidligere tester var basert på tekstinnhold fra lokale tekster, men nå som vi bruker Sanity fungerer de ikke lengre og data-testid brukes for å identifisere komponenter istedenfor.

✅ Checklist

  • 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:

  • Nye tester trengs ikke, men eksisterende tester er endret slik at de fungerer med Sanity tekster istedenfor lokale tekster.
  • En test har blitt fjernet ettersom den var basert lokale tekster og ikke lengre er nødvendig.

🤷‍♀ ️Hvor er det lurt å starte?

  • Alt i ett

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

👀 Screen shots

  • Ingen visuelle endringer

image

Copy link
Contributor

@hanneolsen hanneolsen left a comment

Choose a reason for hiding this comment

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

Det ser bra ut. Jeg fant to TODOer som nok burde sees på, men forhåndsapprover slik at du kan merge når TODOene er ferdig ☺️

….com:navikt/familie-ba-soknad into legger-til-sanity-tekster-i-om-barnet-steget
@ole-kristian-rudjord ole-kristian-rudjord merged commit f9f2e84 into main Dec 13, 2024
5 checks passed
@ole-kristian-rudjord ole-kristian-rudjord deleted the legger-til-sanity-tekster-i-om-barnet-steget branch December 13, 2024 12:59
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