Skip to content

Expressions: Newly hidden fields keep validation messages #617

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.

Sign up for GitHub

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

Jump to bottom
Closed
olemartinorg opened this issue Nov 2, 2022 · 0 comments · Fixed by #627
Closed

Expressions: Newly hidden fields keep validation messages #617

olemartinorg opened this issue Nov 2, 2022 · 0 comments · Fixed by #627
Labels
area/logic related to logic/dynamics/expressions kind/bug Something isn't working org/krt

Comments

@olemartinorg
Copy link
Contributor

olemartinorg commented Nov 2, 2022 •
edited
Loading

Description of the bug

Fields hidden using expressions (#540) may retain their validation messages, if any exists.

Steps To Reproduce

  1. Add logic to show/hide another field
  2. Induce a validation error in the hidable field
  3. Hide the field
  4. Observe the error message still shows up in the bottom error box
@olemartinorg olemartinorg added kind/bug Something isn't working area/logic related to logic/dynamics/expressions org/krt status/triage labels Nov 2, 2022
@olemartinorg olemartinorg moved this to 📈 Todo in Team Apps Nov 2, 2022
@olemartinorg olemartinorg mentioned this issue Nov 2, 2022
Closed
@olemartinorg olemartinorg moved this from 📈 Todo to 👷 In Progress in Team Apps Nov 3, 2022
@olemartinorg olemartinorg self-assigned this Nov 4, 2022
This was referenced Nov 7, 2022
Merged
Closed
@olemartinorg olemartinorg moved this from 👷 In Progress to 🔎 Review in Team Apps Nov 7, 2022
@olemartinorg olemartinorg removed their assignment Nov 7, 2022
Repository owner moved this from 🔎 Review to ✅ Done in Team Apps Nov 8, 2022
@olemartinorg olemartinorg moved this from ✅ Done to 🧪 Test in Team Apps Nov 8, 2022
@olemartinorg olemartinorg moved this from 🧪 Test to ✅ Done in Team Apps Nov 18, 2022
@olemartinorg olemartinorg mentioned this issue Nov 23, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/logic related to logic/dynamics/expressions kind/bug Something isn't working org/krt
Projects
Team Apps
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Removing validation messages when hiding fields using dynamics Altinn/app-frontend-react
1 participant
@olemartinorg