Altinn / app-frontend-react

Altinn application React frontend
BSD 3-Clause "New" or "Revised" License
16 stars 24 forks source link

Summary component: Dynamikk og sporvalg lagt på skjemakomponentene må gjenspeiles i summary #503

Closed rvessb closed 1 year ago

rvessb commented 1 year ago

Description

Dynamikk og sporvalg som er brukt på skjemakomponentene må gjenspeiles i summary-komponentene.

Slik det er i dag må det legges på egen skjul-vis-logikk for summary som gjenspeiler dynamikk og sporvalg i selve skjemaet. Dette medfører en stor ekstra kostnad i utviklingstid og ikke minst testing da man må teste at skjema og oppsummering fungerer likt.

Additional Information

No response

rvessb commented 1 year ago

org/ssb

FTLems commented 1 year ago

Kan dere tagge org/krt her ?

olemartinorg commented 1 year ago

Relevant:

olemartinorg commented 1 year ago

@rvessb @FTLems Da skal fiksen være på plass i siste alpha-versjon av app-frontend - kunne dere testet og gitt tilbakemelding om det fungerer som tiltenkt? :pray:

olemartinorg commented 1 year ago

@rvessb @StianVestli @FTLems @ivarne

Har noen av dere anledning til å teste denne fiksen og si ifra om det fungerer som dere tenkte? :pray: Som nevnt, tilgjengelig i siste alpha. Vi prøver å være litt mer restriktive med nye releaser (siden ting av og til har gått galt), så vi håper å få denne gjennom litt grundigere testing før vi lanserer endringen (selv om vi åpenbart tester ting internt hos oss også).

StianVestli commented 1 year ago

@olemartinorg Skal denne støtte og at hvis en skjuler en hel side ved sporvalg så vil den ikke dukke opp på summaryen? eller er det bare skjul/vis på enkelt komponter som skal støttes?

olemartinorg commented 1 year ago

@StianVestli Det skal fungere også om man skjuler en hel side, ja. :crossed_fingers: En hel side med komponenter som er skjult betyr at alle komponentene derunder er skjult - men bare om man skjuler siden med et uttrykk. Skjuling av en side ved hjelp av sporvalg-funksjonaliteten (også kalt PageOrder på backend) gjør at hele siden må mange måter "forsvinner" for frontend, så da faller den logikken dessverre ut, og paradoksalt nok vil sannsynligvis summary-komponenten da finne kilden og vise innholdet(!).

StianVestli commented 1 year ago

@olemartinorg Det ser ut som det funker som det skal på enkeltfelter , men ikke på helgruppe ser det ut til, og ikke ved sporvalg som du selv sier at det ikke vil funke. Men å skjule en grupper burde funke kanskje?

olemartinorg commented 1 year ago

@StianVestli Superdupert, takk for tilbakemelding! :partying_face: Da henter jeg denne tilbake til tegnebrettet. Enig i at skjuling av en gruppe (og repeterende gruppe) skal skjule komponentene under, og det er noe vi har snakket om før - men som dessverre har falt litt ut. God anledning til å fikse på det nå.

olemartinorg commented 1 year ago

@StianVestli Nå er dette levert (i siste release), men jeg setter ikke oppgaven som ferdig enda, da jeg vil legge til automatiserte tester på dette. Se gjerne over og test ut om det fungerer! (Det vil si, skjuling av en ikke-repeterende gruppe vil nå skjule komponentene inni den.)

StianVestli commented 1 year ago

@olemartinorg Da ser det ut som dette funker hvis jeg har klart å teste riktig.