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

Klage uten brev forberedelse #2913

Merged
merged 8 commits into from
Oct 3, 2024
Merged

Conversation

ViktorGSolberg
Copy link
Contributor

Hvorfor er denne endringen nødvendig? ✨

Dette er egentlig bare en forberedelse til denne oppgaven, som egentlig bare behøver noen små endringer i OpprettBehandlingModal, men jeg synes koden var relativt rotete så jeg tillater meg en liten omskriving før jeg gyver løs på den egentlige oppgaven.

Det som er gjort er:

  • Endring av navn på komponenter, filer og variabler for mer lesbar kode
  • Utrekking av kode til egne komponenter for mer lesbar kode
  • Opprettet en mappe "OpprettBehandling" hvor jeg har plassert OpprettBehandlingModal og alle komponenter den bruker for enklere å kunne finne igjen relevant kode

Ellers synes jeg alle comitt-beskjedene forklarer ganske godt hva de gjør hver for seg.

FØR:
Skjermbilde 2024-10-02 kl  16 16 01
Skjermbilde 2024-10-02 kl  16 15 21
Skjermbilde 2024-10-02 kl  16 15 36
Skjermbilde 2024-10-02 kl  16 15 51

ETTER:
Skjermbilde 2024-10-02 kl  16 16 31
Skjermbilde 2024-10-02 kl  16 16 52
Skjermbilde 2024-10-02 kl  16 17 11
Skjermbilde 2024-10-02 kl  16 17 24

Copy link
Contributor

@charliemidtlyng charliemidtlyng left a comment

Choose a reason for hiding this comment

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

Ser veldig bra ut! 💪

…evurdering-komponenten. Dette blir sjekket lenger opp i komponenthierarkiet
@ViktorGSolberg ViktorGSolberg merged commit 79e358e into main Oct 3, 2024
1 check passed
@ViktorGSolberg ViktorGSolberg deleted the klage-uten-brev-forberedelse branch October 3, 2024 09:18
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