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

Frontend rules are not supported for repeating groups #263

Closed
lorang92 opened this issue Feb 17, 2021 · 7 comments
Closed

Frontend rules are not supported for repeating groups #263

lorang92 opened this issue Feb 17, 2021 · 7 comments
Labels
area/logic related to logic/dynamics/expressions bug/c1 kind/bug Something isn't working org/okokrim org/pod

Comments

@lorang92
Copy link
Contributor

Describe the bug

Frontend rules are not supported for repeating groups.
This can however be solved with backend calculations.

To Reproduce

Steps to reproduce the behavior:

  1. Go to an app with a repeating group
  2. Try adding a rule targeting rep group
  3. See that this is not possible.

Expected behavior

This should work as expected. See implementation for dynamics.

@lorang92 lorang92 added kind/bug Something isn't working area/logic related to logic/dynamics/expressions solution/app-frontend labels Feb 17, 2021
@lvbachmann
Copy link
Contributor

@lorang92 Do you know of any app owner who needs this?

@lorang92
Copy link
Contributor Author

@lvbachmann no, no concrete needs. I suspect rules frontend is not used that much in general, so I wouldn't think this is our highest priority.

@RogerKjaernsrod
Copy link

@lorang92 Er det noen planer om å utføre denne?

@lorang92
Copy link
Contributor Author

@RogerKjaernsrod det blir nok et spørsmål som må gå til PO @FinnurO. Jeg tenker jo dette er en naturlig bit å støtte også i repeterende grupper, spesielt om man nå har et behov for dette.

Kan jo nevne at slik kalkulering også kan løses backend vha. data processing., så man har en fin workaround. Men er bedre ytelse om vi også kan støtte dette frontend.

@RogerKjaernsrod
Copy link

RogerKjaernsrod commented Feb 24, 2022

@lorang92 @TheTechArch Det er jo noen andre issues ifm det å utføre kalkuleringer i backend (#8054), så det er jo ikke en helt fullgod workaround. Men inntil videre så har jeg fått løst det som skal kalkuleres med backend kalkuleringer.
Og angående dette med ytelse så er det jo litt uheldig at man i ProcessDataWrite ikke får noen informasjon om hvilket felt det var som trigget kallet. For slik det er nå så må man kjøre gjennom all kode der, selv om det kanskje kun er koden for feltet som ble endret som er relevant å kjøre.

@haakemon haakemon transferred this issue from Altinn/altinn-studio Jun 28, 2022
@lorang92 lorang92 moved this from 📈 Todo to ⚠ Blocked in Improvements repeating groups Aug 17, 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 12, 2022
@olemartinorg
Copy link
Contributor

Mulig løst via #355 (ny dynamikk via uttrykk, som har støtte for rep.grupper). Denne dynamikken har støtte for vis/skjul, men den har fortsatt ikke støtte for kalkuleringer (men det er planlagt støttet).

@olemartinorg olemartinorg moved this from ⚠️ Blocked to 📈 Todo in Team Apps Oct 27, 2022
@FinnurO
Copy link

FinnurO commented Dec 16, 2022

Lukker fordi kalkulering er eget issue. #646

Repository owner moved this from ⚠ Blocked to 🧪 Testing in Improvements repeating groups Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/logic related to logic/dynamics/expressions bug/c1 kind/bug Something isn't working org/okokrim org/pod
Projects
Status: Done
Archived in project
Status: 🧪 Testing
Development

No branches or pull requests

6 participants