-
Notifications
You must be signed in to change notification settings - Fork 36
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
fix(FormEditor): allow condition="always" #4879
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing instructions
- Make sure there is no validation error for
condition="always"
- Make sure there is no validation error for any valid field name
condition="text1=abc"
- Make sure there are validation errors for any other values
Looks good, error only thrown when there is an invalid field name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing Instructions
- Make sure there is no validation error for
condition="always"
- Make sure there is no validation error for any valid field name
condition="text1=abc"
- Make sure there are validation errors for any other values
Looking good on my end--tested out a handful of different values for ones that should be invalid (random letters, properties that are used in different fields, invalid field names, true/false) and didn't run into any issues. Good work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Make sure there is no validation error for
condition="always"
- Make sure there is no validation error for any valid field name
condition="text1=abc"
- Make sure there are validation errors for any other values
I am now able to save my original form that contains the conditional example 👍👍
@CarolineDenis can this be merged? |
no, this is part of 9.6 we have 9.5 before |
didn't conditional form editing go into the 7.9.4 release? |
Fixes #4872
Checklist
and self-explanatory (or properly documented)
Testing instructions
condition="always"
condition="text1=abc"