Skip to content
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

Sletting av data på skjemaelementer som skjules vha rendringsregel #7849

Closed
rvessb opened this issue Jan 19, 2022 · 5 comments
Closed

Sletting av data på skjemaelementer som skjules vha rendringsregel #7849

rvessb opened this issue Jan 19, 2022 · 5 comments
Labels
kind/feature-request Feature request that will be considered. org/krt Issues relevant for Finanstilsynet org/okokrim Issues relevant for Økokrim org/ssb Issues relevant for Statistisk sentralbyrå. team/apps

Comments

@rvessb
Copy link

rvessb commented Jan 19, 2022

What needs to be solved?

Å unngå å måtte lage mange ekstra skreddersydde regler/kalkuleringer for å slette data på skjemaelementer som skjules vha renderingsregler. I skjema med mye dynamikk/sporvalg er man avhengig av å kunne gjøre dette for at oppgavegiver skal kunne "rygge" i under skjemabesvarelse.

How do you want it solved?

Som et tilvalg i selve renderingsregelen: "slett data på alle komponentene når de skjules". Slik at data slettes på bakgrunn av om feltene skjules, uavhengig av hvilken type handling (SHOW/HIDE) som er satt på regelen. Her må man ta høyde for at "komponent" kan være en gruppe som igjen inneholder flere skjemaelementer.

Skjermbilde_slettdata

Alternative solutions

Additional context

@rvessb rvessb added the kind/feature-request Feature request that will be considered. label Jan 19, 2022
@altinnadmin altinnadmin assigned FinnurO and unassigned altinnadmin Jan 26, 2022
@FinnurO FinnurO added the org/ssb Issues relevant for Statistisk sentralbyrå. label Jan 26, 2022
@RogerKjaernsrod RogerKjaernsrod added the org/okokrim Issues relevant for Økokrim label Feb 24, 2022
@FTLems
Copy link

FTLems commented May 10, 2022

Kan dere legge til org/krt label på denne saken?

@FinnurO FinnurO added the org/krt Issues relevant for Finanstilsynet label May 10, 2022
@FinnurO FinnurO moved this to Todo in Team Studio (old) May 10, 2022
@FinnurO FinnurO removed their assignment Oct 6, 2022
@rvessb rvessb added this to Issues SSB Oct 7, 2022
@rvessb rvessb moved this to Todo in Issues SSB Oct 10, 2022
@olemartinorg olemartinorg moved this from Todo to In Progress in Issues SSB Nov 2, 2022
@olemartinorg
Copy link
Contributor

@rvessb Det er ikke helt den løsningen dere ba om, men dette er nå på sett og vis mulig ved bruk av nye dynamiske uttrykk. Data kan nå settes opp til å slettes fra datamodellen ved innsending (ikke under utfylling). Det vil si, man kan velge at alle skjulte data skal fjernes fra datamodellen. Se kommentaren til Ivar her for detaljer. Merk at data som ligger i datamodellen som aldri var knyttet til en komponent ikke blir slettet.

Ser dere over om dette løser samme utfordring?

@rvessb
Copy link
Author

rvessb commented Nov 24, 2022

@olemartinorg Har sett på det Ivar skrev. Veldig fint at han fikk "panikk" og tok med mulighet for å slå denne funksjonaliteten av og på, for vi kan nok ikke ha dette påslått på alle våre skjema (det beste hadde vært å hatt "bryteren" hadde vært ned på komponentnivå).
Er det slik å forstå at data brukt på sider som er skjult via "pageOrder-logikk" også slettes (ikke bare skjul via de nye "Dynamiske uttrykk") ? I så fall tror jeg vi sier at behovet beskrevet i denne saken er dekt..

@ivarne
Copy link
Member

ivarne commented Nov 25, 2022

Siden du kan skjule hele sider med ny dynamikk, har jeg ikke implementert støtte for IPageOrder.

Det vil være en enkel endring å støtte konfigurasjon for om enkelt komponenter/grupper/sider skal være med i logikken for å slette felt. Kan du prøve å forklare mer hvorfor dette er et behov? Konfigurasjon øker drastisk risiko for feil (i altinn koden), og bør helst ikke legges inn om det ikke er en god grunn.

@olemartinorg
Copy link
Contributor

olemartinorg commented Dec 27, 2022

Lukker denne saken, og anser utfordringen som løst ved hjelp av dynamikk og støtte for automatisk sletting av data fra datamodellen når skjemafelter er skjult. (Se info om RemoveHiddenDataPreviewdenne siden).

Som diskutert med SSB, ønsker vi neppe å legge til støtte for dette per-komponent, da det fort kan bli veldig uoversiktlig. Flere komponenter kan peke mot samme sted i datamodellen, og man ville fort hatt behov for verktøy for å visualisere hva som kan føre til at data blir slettet fra datamodellen og ikke. For andre ting man putter i datamodellen blir det nok mer aktuelt med hjelpefelter/skygge-datamodell (se Altinn/app-lib-dotnet#560 og Altinn/app-frontend-react#744).

Rop ut om jeg var for kjapp på labben her, eller meld inn inn et alternativt issue dersom dette ønskes (men unngå gjerne spesifikke forslag til hvordan dette ønskes implementert - da dette issuet lukkes fordi vi ikke ønsker å implementere sjekkboksen som er tegnet opp og ønsket i første post her).

@github-project-automation github-project-automation bot moved this from Todo to Done in Team Studio (old) Dec 27, 2022
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues SSB Dec 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature-request Feature request that will be considered. org/krt Issues relevant for Finanstilsynet org/okokrim Issues relevant for Økokrim org/ssb Issues relevant for Statistisk sentralbyrå. team/apps
Projects
Archived in project
Status: Done
Status: Done
Development

No branches or pull requests

7 participants