Journalføring 2.0 bruker avsender #2591
Conversation
…ng av fylt/tomt symbol. Skal kunne kopiere personident fra avsender panel dersom bruker er avsender
| const [harRedigertAvsender, settHarRedigertAvsender] = useState<boolean>(false); | ||
| const [nyAvsender, settNyAvsender] = useState<string>(''); |
There was a problem hiding this comment.
Må vi oppdatere avsender i staten slik at vi "sender inn" avsender også?
There was a problem hiding this comment.
Gikk nok litt fort i svingene her, ja
There was a problem hiding this comment.
Fixd 317b507
| <Tittel size={'small'} level={'2'}> | ||
| Avsender | ||
| </Tittel> | ||
| <AvsenderPanel journalpostResponse={journalResponse} /> |
There was a problem hiding this comment.
Kanskje sende med journalpostState her og oppdatere avsender i staten?
There was a problem hiding this comment.
Fixd 317b507
…nding til backend
| @@ -106,6 +108,7 @@ export const useJournalføringState = ( | |||
| const [valgtDokumentPanel, settValgtDokumentPanel] = useState<string>( | |||
| utledFørsteDokument(journalResponse.journalpost.dokumenter) | |||
| ); | |||
| const [nyAvsender, settNyAvsender] = useState<string>(''); // TODO: Denne må sendes med til backend for å bli satt | |||
There was a problem hiding this comment.
Tror vi må tenke litt her - for det er to-tre scenarier:
- Ingen endring på avsender, da vil vi ikke sende inn noe
- Avsender er bruker (endring) -> sende inn et eller annet...
- Avsender er IKKE bruker -> sende inn navn...
There was a problem hiding this comment.
Ja, men dette er vel logikk som vil bli lagt inn i metoden for fullførJournalføringV2()?
Send inn dersom nyAvsender ikke er tom streng og ikke er det samme før? Eventuelt kan man sende inn dette uansett og la backend utlede om man skal sette ny avsender eller ikke - basert på de samme reglene
There was a problem hiding this comment.
m.a.o. dette vil bli lagt til i PR for innsending av data til backend
| const [erBrukerAvsender, settErBrukerAvsender] = useState<boolean>(false); | ||
| const [harRedigertAvsender, settHarRedigertAvsender] = useState<boolean>(false); | ||
|
|
||
| const avsender = utledAvsender(erBrukerAvsender, harRedigertAvsender, nyAvsender, navn); |
There was a problem hiding this comment.
Avsender ligger nesten alltid på journalposten fra før - tror kanskje dét bør være "default" verdi ved journalføring - og heller mulig å endre dersom det er "feil" ?
There was a problem hiding this comment.
fixd a1124f9
…r. Dette kan saksbehandler endre ved å huke av for at bruker ikke er avsender
|
|
||
| const IconContainer = styled.div` | ||
| color: ${ABlue500}; | ||
| `; |
There was a problem hiding this comment.
Kan IconContainer tas i en egen komponent? Det brukes også Ikon- og IconContainer om hverandre
There was a problem hiding this comment.
Hmmm, jeg vil si at i dette tilfellet blir det litt overkill å lage en egen .tsx fil for å flytte ut tre kodelinjer, men jeg er enig i at navngivningen burde være mer konsekvent så det skal jeg fikse 👍
| </ExpansionCardHeader> | ||
| <ExpansionCard.Content> | ||
| <ExpansionCardContent> | ||
| <Checkbox |
There was a problem hiding this comment.
Er denne greit alignet med raden over ? (ref screenshot)
There was a problem hiding this comment.
Det ser kanskje litt off ut basert på bildet over, men sett sammen med de andre panelene som også opptrer på siden så ser det nok mer naturlig ut
|
Kudos, SonarCloud Quality Gate passed!
|
Hvorfor er denne endringen nødvendig? ✨
Skal løse følgende punkter fra favro-kort
Vise brukers nanv, id og kopifunksjon i panel for bruker.
Brukerpanelet skal ikke kunne åpnes
Visning av avsenders navn, id og kopifunksjon i avsenderpanel
Hvis avsender ikke er sat skal det stå "Ukjent avsender"
Avsender panel skal kunne ekspanderes for avhuking av om avsender er bruker og for manuell inntasting av avsenders navn/bedrift.
Følger: