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: støtte ny implementasjon av flettefelt #1129

Merged
merged 2 commits into from
Mar 13, 2024

Conversation

idaame
Copy link
Contributor

@idaame idaame commented Mar 13, 2024

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

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

Legger til støtte for ny implementasjon av flettefelt. I en mellomfase vil koden nå støtte både gammel og ny variant, frem til alt gammelt er slettet fra Sanity. Tanken er at BA-tekster ikke skal bruke de gamle flettefeltene, men det er greit å beholde den gamle implementasjonen enn så lenge, just in case.

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

Er det noe du er bekymret eller usikker på? Beskriv det gjerne her.
Kommer til å legge til tilsvarende i ks-soknad, slik at denne også støtter begge type flettefelter i en mellomperiode.

✅ 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:
Har testet manuelt at det funker

🤷‍♀ ️Hvor er det lurt å starte?

F.eks. commit for commit, alt i ett?
Alt er i samme commit

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei, men kan også forklare muntlig om det er ønskelig

👀 Screen shots

Har det visuelle endret seg? Legg til før- og etterbilder!
Har testet å legge til det nye flettefeltet i en test-tekst. Har deretter testet å legge inn denne teksten i koden, og rendret ut både som TekstBlock-komponent og brukt stringen fra plainTekst-funksjonen. Hvor jeg har puttet inn komponentene gir ikke nødvendigvis mening, men teksten som rendres er riktig:)

Fra plainTekst-funksjon:
image

Fra TekstBlock-funksjon:
image

@idaame
Copy link
Contributor Author

idaame commented Mar 13, 2024

Sanity-implementasjon: navikt/familie-baks-soknad-sanity#467

Copy link
Contributor

@kristeine kristeine 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

@bragejahren bragejahren left a comment

Choose a reason for hiding this comment

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

Det var en veldig mye mindre endring enn jeg hadde sett for meg 😝

@idaame idaame enabled auto-merge (squash) March 13, 2024 14:35
@idaame idaame merged commit 930ae9f into main Mar 13, 2024
5 checks passed
@idaame idaame deleted the ny-flettefelt-implementasjon branch March 13, 2024 14:53
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