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

Repeating groups: Kunne slå av muligheten til å fjerne og legge til grupper #64

Closed
3 tasks
Febakke opened this issue Apr 27, 2022 · 7 comments
Closed
3 tasks
Assignees
Labels
kind/user-story Used for issues that describes functionality for our users. org/ssb Issues relevant for Statistisk sentralbyrå.

Comments

@Febakke
Copy link
Member

Febakke commented Apr 27, 2022

Description

Noen grupper skal ikke kunne fjernes. Det vil typisk være pre-utfylte grupper som må være med i innsendingen. Vi må ha mulighet til å skjule slett-funksjonaliteten for disse elementene. Dette må være mulig å sette på noen eller alle elementer i den repeterende gruppen

Considerations

Finnes det caser det man også skal kunne skjule "legg til ny" funksjonaliteten?

Ops requirements

Acceptance criteria

  • Man kan fjerne "slett" knapp på pre-utfylte elementer
  • Hvis brukeren kan legge til egne grupper så kan disse fjernes

Specification tasks

  • Forstår brukeren hvorfor den ikke kan slettes?
    - Her må vi se på om vi visuelt burde skille mellom preutfylte og egenlagde grupper for å gjøre det tydeligere at det er en forskjell på dem

Development tasks

@Febakke Febakke added the kind/user-story Used for issues that describes functionality for our users. label Apr 27, 2022
@Halland67
Copy link

org/ssb

@olemartinorg
Copy link
Contributor

Regner dette som en utvidelse av eksisterende funksjonalitet som gjør det mulig å skru av "legg til ny" for en gruppe:

Det er altså mulig å fjerne ny-knappen (lenke til dokumentasjon) og slett-knappen (lenke til dokumentasjon), men det er nok ikke mulig å gjøre dette for spesifikke "låste" rader i gruppen. Det vil si, man kan ikke legge til pre-utfyllte rader i en gruppe som ikke kan slettes, og samtidig beholde funksjonaliteten for å legge til flere rader manuelt (som deretter kan slettes igjen).

For å få til denne nye funksjonaliteten regner jeg med vi må bruke et hjelpefelt i datamodellen som spesifiserer om raden kan slettes eller ikke, og spesifisere hvilket felt i datamodellen som bestemmer dette i gruppens skjema-oppsett.

@Halland67 og @rvessb: For vår egen prioritering kunne det være greit for oss å vite om behovene deres blir løst av eksisterende funksjonalitet, eller om dere trenger støtte for å markere hver enkelt rad som mulig å slette eller ikke.

@olemartinorg olemartinorg removed their assignment Jul 29, 2022
@olemartinorg
Copy link
Contributor

En liten oppdatering. Jeg begynner å se litt på #61, som også er et prioritert issue. I forbindelse med den saken må jeg implementere det meste som trengs for å løse det som gjenstår i denne saken også, så da kan jeg like godt få med dette samtidig. Jeg lager en epic, #355, og har samlet disse sakene der.

@rvessb
Copy link

rvessb commented Aug 11, 2022

@olemartinorg Vi har også tenkt at men må bruke et hjelpefelt i datamodellen for å si hvilken "rader" i repeterende gruppe man skal få lov til å slette og ikke. Det vi nok også vil ha behov for er å skrivebeskytte enkelte av feltene som inngår i "repeterende gruppen" dersom de er "preutfylte", dette kan da også støtte seg på samme hjelpefelt.
I utgangspunktet tenker jeg at det ikke er behov for å merke hvilken rader som det ikke er mulig å slette, utover at sletteknappen er deaktivert.

@lorang92 lorang92 moved this from 📈 Todo to ⚠ Blocked in Improvements repeating groups Aug 17, 2022
@elwal elwal added this to Issues SSB Oct 3, 2022
@elwal elwal moved this to Todo in Issues SSB Oct 5, 2022
@Magnusrm Magnusrm moved this to 📈 Todo in Team Apps Oct 12, 2022
@Magnusrm Magnusrm moved this from 📈 Todo to ⚠️ Blocked in Team Apps Oct 24, 2022
@olemartinorg olemartinorg moved this from ⚠️ Blocked to 📈 Todo in Team Apps Oct 27, 2022
olemartinorg pushed a commit that referenced this issue Nov 2, 2022
* Add explanation of the keyword 'today'

* Update schemas/json/layout/layout.schema.v1.json

Co-authored-by: Steffen Lorang Ekeberg <steffen.ekeberg@gmail.com>

* Fix description after feedback on PR

* Remove extra quote character

Co-authored-by: Steffen Lorang Ekeberg <steffen.ekeberg@gmail.com>
@olemartinorg olemartinorg moved this from Todo to In Progress in Issues SSB Nov 2, 2022
@bjosttveit bjosttveit self-assigned this Nov 17, 2022
@bjosttveit bjosttveit moved this from 📈 Todo to 👷 In Progress in Team Apps Nov 17, 2022
@olemartinorg
Copy link
Contributor

@bjosttveit FYI, the remaining part of this task now basically revolves around this TODO from #540:

deleteButton?: ExpressionOr<'boolean'>; // TODO: Make expressions resolve per-row

@bjosttveit
Copy link
Member

Yeah, I figured I would give it a try. I need to get more familiar with expressions anyway

@bjosttveit bjosttveit moved this from 👷 In Progress to 📈 Todo in Team Apps Nov 17, 2022
@bjosttveit bjosttveit assigned bjosttveit and unassigned bjosttveit Nov 21, 2022
@bjosttveit bjosttveit moved this from 📈 Todo to 👷 In Progress in Team Apps Nov 23, 2022
@bjosttveit bjosttveit removed their assignment Nov 23, 2022
@bjosttveit bjosttveit moved this from 👷 In Progress to ⚠️ Blocked in Team Apps Nov 23, 2022
@tjololo
Copy link
Member

tjololo commented Nov 24, 2022

Solved through #61

@tjololo tjololo moved this from ⚠️ Blocked to 📈 Todo in Team Apps Nov 24, 2022
@Magnusrm Magnusrm moved this from 📈 Todo to 👷 In Progress in Team Apps Nov 29, 2022
@Magnusrm Magnusrm moved this from 👷 In Progress to 📈 Todo in Team Apps Nov 29, 2022
@Magnusrm Magnusrm moved this from 📈 Todo to 🧪 Test in Team Apps Dec 13, 2022
@Magnusrm Magnusrm moved this from 🧪 Test to ✅ Done in Team Apps Dec 15, 2022
Repository owner moved this from Todo to Done in Team Studio (old) Dec 15, 2022
Repository owner moved this from Todo to Done in Layout expressions Dec 15, 2022
Repository owner moved this from In Progress to Done in Issues SSB Dec 15, 2022
Repository owner moved this from ⚠ Blocked to 🧪 Testing in Improvements repeating groups Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/user-story Used for issues that describes functionality for our users. org/ssb Issues relevant for Statistisk sentralbyrå.
Projects
Archived in project
Status: Done
Archived in project
Status: Done
Status: 🧪 Testing
Development

No branches or pull requests

8 participants