Skip to content

Bytt ut RhfRadioGroup med RhfRadioGroupNew #6546

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

Merged
merged 3 commits into from
Aug 15, 2025
Merged

Bytt ut RhfRadioGroup med RhfRadioGroupNew #6546

merged 3 commits into from
Aug 15, 2025

Conversation

tor-nav
Copy link
Collaborator

@tor-nav tor-nav commented Aug 13, 2025

Kommentar til @sirimykland : Ser det er to versjonar av TrueFalseInput.tsx. Eg er usikker på om me treng denne wrapparen... den wrappar jo litt logikk så kanskje greit å ha ein komponent som definerar bruken av true/false-radio og bruk av children kobla til dei ulike valga? Bør den i tilfelle leggast i form-hooks?

@tor-nav tor-nav requested a review from a team as a code owner August 13, 2025 06:49
@tor-nav tor-nav requested a review from sirimykland August 13, 2025 06:53
Copy link

@sirimykland
Copy link
Contributor

Kommentar til @sirimykland : Ser det er to versjonar av TrueFalseInput.tsx. Eg er usikker på om me treng denne wrapparen... den wrappar jo litt logikk så kanskje greit å ha ein komponent som definerar bruken av true/false-radio og bruk av children kobla til dei ulike valga? Bør den i tilfelle leggast i form-hooks?

Jeg lagde komponenten først når jeg refakturerte papirsøknad-ui-komponenter fordi jeg så at det var veldig mye duplisert kode(19 tilfeller i papirsøknad-ui-komponenter).

I tillegg kom den til nytte i fakta-omsorg-og-rett panelet, og siden vi ikke har noen logisk pakke å legge en i fp-frontend så ble den duplisert.

Jeg er enig i at den kan flyttes til ft-form-hooks 😄

@tor-nav tor-nav merged commit 92b2de2 into master Aug 15, 2025
7 checks passed
@tor-nav tor-nav deleted the radiogroupnew branch August 15, 2025 10:34
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