-
Notifications
You must be signed in to change notification settings - Fork 2
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
Manglende gruppeId under tenor/test-norge search flyten #3764
base: master
Are you sure you want to change the base?
Conversation
- Viser riktig gruppe i bestillingsHeader, dersom den mangler fra opts #deploy-test-frontend
- Lagt til BestillingsveilederContextType for bedre typescript support #deploy-test-frontend
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mulig jeg har gått glipp av noe her, men trenger vi et helt eget steg for å velge gruppe? Kan vi ikke bare legge gruppe-valg inn i modalen man uansett får opp når man importerer person? 🤔
Vi tenkte vel at modalen egentlig kan fjernes og at det steget gir flere fordeler; blant annet at man kan gå tilbake og velge mal/gruppe istedenfor at det er et "endelig valg" hvor man må avbryte bestillingen for å gjøre endring 😅 |
Åja, har folk klaget på dette? 😅 Det blir jo akkurat samme greia når man bare oppretter en vanlig person også da 😬 For da synes jeg i så fall vi må være konsekvente, og fjerne all bruk av modal før bestilling/import, og legge alt inn i steg 0. Blir litt forvirrende når flyten blir helt ulik 🤔 |
Ingen klager så vidt jeg vet, men har jo merket selv at dersom man velger feil mal fordi to maler er ganske like, så er det jo 5 klikk for å velge den andre igjen, selv om man finner ut av det på steg 1 allerede 🤪 Ja, den også bør flyttes til nytt steg, men vi tenkte å vente til du var tilbake før jeg gjorde store endringer på standard oppsettet 😂 |
Ah, ok, siden PRen var klar for review trodde jeg den var ferdig 😅 |
Det er foreløpig bare gjort endringer i tenor/testnorge biten for å få det til å fungere igjen med valg av gruppe, kan forsåvidt ta diskusjonen rundt det på fredagsmøtet etterpå 🙂 |
Ok, for da er jeg tilbake til at jeg helst bare ville hatt det valget i modalen (midlertidig), så slipper vi en lengre periode hvor alt ser ulikt ut 🙂 |
Ja, høres bra ut det altså 👍 |
# Conflicts: # apps/dolly-frontend/src/main/js/package-lock.json # apps/dolly-frontend/src/main/js/src/components/bestillingsveileder/stegVelger/steg/steg2/Steg2.tsx
…håndterer dette - Skrevet om mye funksjonalitet rundt maler og gruppeId handling
#deploy-test-frontend
…g av gruppe for bestilling #deploy-test-frontend
#deploy-test-frontend
…, Test-norge soek, validering og noen andre mindre komponenter - Oppdatering og omskriving av Playwright tester #deploy-test-frontend
…jonalitet #deploy-test-frontend
This pull request includes several updates to the
Bestillingsveileder
components to enhance the handling ofgruppeId
by incorporating thereact-hook-form
library. The changes ensure that thegruppeId
is consistently watched and updated across different components.Key changes include:
Integration with
react-hook-form
:apps/dolly-frontend/src/main/js/src/components/bestillingsveileder/BestillingsveilederHeader.tsx
: AddeduseFormContext
to watch and utilizegruppeId
from the form context, ensuring the correctgruppeId
is used when fetching the group details.apps/dolly-frontend/src/main/js/src/components/bestillingsveileder/stegVelger/steg/steg1/paneler/Personinformasjon.tsx
: IncorporateduseFormContext
to watchgruppeId
and use it when fetching identifiers related to the group. [1] [2]apps/dolly-frontend/src/main/js/src/components/bestillingsveileder/stegVelger/steg/steg3/VelgGruppe.tsx
: Updated theuseEffect
hook to useformMethods.watch
forgruppeId
instead offormMethods.getValues
to ensure real-time updates.Consistent
gruppeId
usage:apps/dolly-frontend/src/main/js/src/components/bestillingsveileder/BestillingsveilederHeader.tsx
: Modified the display value for the group title to prioritizeformGruppeId
overopts.gruppeId
for accurate representation.