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

Adding a "validation" trigger on a group component ignores "component"-specific validaitons #269

Closed
lorang92 opened this issue Nov 3, 2021 · 2 comments
Labels
bug/c1 kind/bug Something isn't working

Comments

@lorang92
Copy link
Contributor

lorang92 commented Nov 3, 2021

Describe the bug

Setting the validate flag in triggers on a group will run data model/required validations but seems to ignore component-specific (defined in formlayout) validations.

To Reproduce

Steps to reproduce the behavior:

  1. Add a group with a date picker component
  2. Add a max date for the component
  3. Use the keyboard to fill out a date later than max
  4. Save the group
  5. See that group saves

Expected behavior

User should be prevented from saving when saving and validateFormComponentsForLayout should validate correctly.

@lorang92 lorang92 added kind/bug Something isn't working solution/app-frontend labels Nov 3, 2021
@lvbachmann
Copy link
Contributor

Must be bumped to B severity if back-end validations doesn't solve the issue.

@haakemon haakemon transferred this issue from Altinn/altinn-studio Jun 28, 2022
@olemartinorg olemartinorg moved this to 📈 Todo in Team Apps Oct 18, 2022
@bjosttveit bjosttveit self-assigned this Nov 3, 2022
@bjosttveit bjosttveit moved this from 📈 Todo to 👷 In Progress in Team Apps Nov 3, 2022
@bjosttveit
Copy link
Member

Not relevant anymore, focusing on #600 instead.

Repository owner moved this from 👷 In Progress to ✅ Done in Team Apps Nov 4, 2022
@bjosttveit bjosttveit removed their assignment Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/c1 kind/bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

4 participants