Tsf 3739 send brev til tredjepart #5276
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…sningDto.java Legger til property ektefelle i PersonopplysningerBasic, og endre property statsborgerskap til å inkludere navn property. Dette slik at den stemmer betre med det backend definerer og returnerer.
Kan no sjå komponenten i meir reelle situasjoner i storybook, med data henta frå Q.
Denne førte til at test av ny kode for sending av brev til tredjepart feila, pga at ein bruker ein form prop til både ein select og ein input der, ut frå andre valg. Når ein då satte verdi gjennom tekst input logga select komponent advarsel om at satt verdi ikkje fantes som option verdi. Dette førte igjen til at tester av dette feila. Test bør fungere mest mulig likt reell bruk av koden, og viss noko skal feile bør det kastast error, eller i alle fall logge som error. Ein advarsel er spesifikt ikkje meint å føre til fatal feil som det her gjorde.
I panelet for sending av brev er der no ein ny avkryssingsboks for "Send til tredjepartsmottaker". Denne visast når valgt brevmal returnert frå backend har prop støtterTredjepartsmottaker == true. Når bruker aktiverer denne avkrysning visast eit input felt for organisasjonsnr. Bruker kan deretter taste inn eit organisasjonsnr som brevet skal sendast til, istadenfor å velge det frå forhåndsdefinerte mottakere i nedtrekkslista. Når bruker fyller inn organisasjonsnr søkast namnet til organisasjonen opp og visast i panelet. Dette søket går foreløpig direkte til Brønnøysundregisteret sitt opne api, men skal endrast til å bruke eit nytt endepunkt på k9-sak backend. Foreløpig funksjonalitet er lagt inn så vi kan starte litt testing i Q uten å vente på backend implementasjon. Når bruker har fyllt inn tredjepartsmottaker og trykker send blir det utfyllte organisasjonsnr sendt til backend på samme måte som viss bruker valgte førehandsdefinert mottaker i nedtrekksliste.
Tillater denne midlertidig slik at ny funksjonalitet for sending av brev til tredjepart kan testast i Q før eigen api backend for søk i enhetsregisteret er implementert.
Text selector had not been changed after gui text changed.
| async getTredjepartsmottakerInfo(orgnr: string) { | ||
| return {name: `Test Org navn (${orgnr})`} | ||
| } | ||
| } satisfies BackendApi |
There was a problem hiding this comment.
satisfies var ny for meg. Virker kult!
There was a problem hiding this comment.
Ja, kan hjelpe til å lage betre intellisense/typesikkerhet i nokre tilfeller, såvidt eg veit.
hallvardastark
approved these changes
Nov 22, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
GUI for sending av brev til tredjepart lagt til.
I panelet for sending av brev er der no ein ny avkryssingsboks for "Send til tredjepartsmottaker". Denne visast når valgt brevmal returnert frå backend har prop støtterTredjepartsmottaker == true.
Når bruker aktiverer denne avkrysning visast eit input felt for organisasjonsnr. Bruker kan deretter taste inn eit organisasjonsnr som brevet skal sendast til, istadenfor å velge det frå forhåndsdefinerte mottakere i nedtrekkslista.
Når bruker fyller inn organisasjonsnr søkast namnet til organisasjonen opp og visast i panelet.
Dette søket går foreløpig direkte til Brønnøysundregisteret sitt opne api, men skal endrast til å bruke eit nytt endepunkt på k9-sak backend. Foreløpig funksjonalitet er lagt inn så vi kan starte litt testing i Q uten å vente på backend implementasjon.
Når bruker har fyllt inn tredjepartsmottaker og trykker send blir det utfyllte organisasjonsnr sendt til backend på samme måte som viss bruker valgte førehandsdefinert mottaker i nedtrekksliste.