Conversation
…te for å fylle ut svar og begrunnelse om delvilkår.
…fra hook til å vurdere om 'Gjenbruk' knapp skal vises.
…erten er nå lagt inn i redigeringsmodus når gjenbruk blir trykket på. Fjernet ubrukt kode.
…rGjenbruk' ikke blir gjort på 'Tidligere vedtaksperioder' og 'Aktivitet'.
…ra før skal gå automatisk. Hvis det er fylt inn noe på et vilkår åpnes en modal hvor man må bekrefte av man ønsker å gjenbruke.
gunnsteingarmo
requested review from
charliemidtlyng,
ViktorGSolberg,
olekvernberg,
ma10s,
throndi and
EndreHM
There was a problem hiding this comment.
Dette er en ny feature som passer perfekt med "feature toggles".
Det gjør hele test og utrullingsprosessen lettere. Da kan vi merge rett i master uten å måtte skru på i prod. Slik det er nå må brancher deployes manuelt til preprod for testing og det er litt mer humpete for fremdriften enn å lage en toggle, rulle ut og slette toggle + kode når det prodsettes.
...frontend/Komponenter/Behandling/Inngangsvilkår/InngangsvilkårHeader/InngangsvilkårHeader.tsx
Show resolved
Hide resolved
src/frontend/Komponenter/Behandling/Vurdering/VisEllerEndreVurdering.tsx
Outdated
Show resolved
Hide resolved
…uker oppdaterInngangsvilkårMedVurdering og setter feilmelding. Må ha med InngangsvilkårHeader, skal fortsatt kunne oppdatere opplysninger og lukke/åpne alle vurderinger.
…y property 'kanGjenbrukes' på hver vurdering.
…es. Bruker hooken i BehandlingContext. Sjekker om en ID til en vilkårsvurdering finnes i listen for gjenbrukbare vilkår.
src/frontend/App/hooks/useHentAlleGjenbrukbareVilkårsvurderinger.ts
Outdated
Show resolved
Hide resolved
src/frontend/App/hooks/useHentAlleGjenbrukbareVilkårsvurderinger.ts
Outdated
Show resolved
Hide resolved
src/frontend/App/hooks/useHentAlleGjenbrukbareVilkårsvurderinger.ts
Outdated
Show resolved
Hide resolved
| ); | ||
| }; | ||
|
|
||
| export default ModalGjenbrukVilkårsvurdering; |
There was a problem hiding this comment.
Jeg er egentlig blitt fan av å exportere direkte der konstanten blir definert (named export) i stedet for å ta i bruk default exports. Grunnen er at når man skal sjekke hvor en komponent som er tatt i bruk så dukker defualt exporten opp i søket og skaper "støy".
Såvidt jeg har forstått så finnes det egentlig ingen god grunn til å bruke default exports heller. Jeg har derfor lyst til at vi skal gå bort fra default exports til named exports generelt i kodebasen
Hvorfor er denne endringen nødvendig? ✨
Det skal være mulig å kun gjenbruke ett bestemt inngangsvilkår i stedet for alle sammen.
Backend PR
Favro