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.

Sign up for GitHub

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

Jump to bottom

Botype annet #1511

Open

toresbe wants to merge 3 commits into master from botype-annet

Contributor

toresbe commented Sep 9, 2024

Implementasjonen av Bosituasjon var problematisk på flere måter:

Primær og sekundær botypeliste hadde forskjellig state - like greit å bare bruke samme tilstandsvariabel
Mutering i setBotype ble ikke avventet - åpnet for synkroniseringsbugs
Komponent for antall ble rendret før defaultValue var klar, førte til at tekstboksen ikke ble initialisert

toresbe added 3 commits

September 9, 2024 13:51


          prelim refaktor av bosituasjon for å fikse "annet"

d273b1a


          Tweaker litt på AntallPersoner

b2c62e5


          Gjør koden mer robust

06ede7e

toresbe requested a review from a team as a code owner

September 9, 2024 12:40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet