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

Bytter til Aksel-komponent FileUpload #1537

Merged
merged 38 commits into from
Mar 17, 2025

Conversation

ole-kristian-rudjord
Copy link
Contributor

@ole-kristian-rudjord ole-kristian-rudjord commented Mar 10, 2025

Favro: NAV-20221
Aksel-komponent: FileUpload
Unleash toggle: familie-ba-soknad.bruk-ny-last-opp-vedlegg-komponent
Samme oppgave er gjort i KS: navikt/familie-ks-soknad#840

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

  • Bytter fra å bruke en egen komponent til å laste opp filer, til å bruke Aksel sin FileUpload komponent.
  • Legger til Sanity tekster knyttet til den nye løsningen.
  • Legger til toggle for å bytte mellom gammel og ny løsning.

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

  • Det er viktig at den nye løsningen testes godt før den tas i bruk i prod. Ikke skru på toggle i prod før den er testet og godkjent av funksjonell test.

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

  • Jeg er usikker på om skrevne tester vil fungere bra for å teste funksjonalitet av komponenten.

🤷‍♀ ️Hvor er det lurt å starte?

  • Alt i ett.
  • Test følgende:
    • Trykk på dropzone feltet og velg filer som skal lastes opp.
      image
      • Kun gyldige fil-typer (.png, .jpg, .jpeg, .pdf) skal være mulig å velge i fil-velgeren som åpnes.
        image
    • Dra og slipp filer i dropzone feltet.
      • Alle fil-typer kan slippes i dropzone feltet.
      • Fil-typer som ikke støttes og filer med andre feil vil legges under "Vedlegg med feil".
        image
    • Velg filer som er for store. Maksgrensen på én fil er 10mb. Det skal føre til følgende feil:
      image
    • Velg filer som har for små dimensjoner. Minimum dimensjoner på én fil er 400x400px. Det skal føre til følgende feil:
      image
    • Velg for mange filer. Maks er 25 filer. Det skal føre til følgende feil:
      image
    • Blokker nettverkskall slik at det oppstår en feil under opplastingen av de neste filene du velger.
      image
      Det skal føre til følgende feil:
      image
      Deretter fjern blokkeringen og trykk på "retry" knappen slik at vedlegget lastes opp på nytt. Det skal føre til at filen ligger under "vedlegg":
      image

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

👀 Screen shots

Før

image

Etter

image

Før - mens filer er under opplastning

Skjermen blir blurry og brukeren kan ikke gjøre noe annet enn å vente.
image

Etter - mens filer er under opplastning

"Laster opp" status legges til på filer under opplastning mens brukeren kan gjøre andre ting i mellomtiden.
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 har testet dette lokalt på min maskin og det fungerer som det skal hos meg 💯

Copy link
Contributor

@GardOS GardOS left a comment

Choose a reason for hiding this comment

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

@ole-kristian-rudjord ole-kristian-rudjord merged commit 7b36578 into main Mar 17, 2025
4 checks passed
@ole-kristian-rudjord ole-kristian-rudjord deleted the feat/bytter-til-aksel-komponent-fileupload branch March 17, 2025 09:31
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.

3 participants