-
Notifications
You must be signed in to change notification settings - Fork 24
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 plugin validators being AND-ed instead of OR-ed in the backend validation. #4136
Fix plugin validators being AND-ed instead of OR-ed in the backend validation. #4136
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.
I'm wondering why it started appearing in #4121
I had the wrong issue number, should have been 4131 |
The frontend applies OR-logic when evaluating validation plugins on a Formio component, which the backend did not replicate correctly.
61eb2be
to
107845c
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4136 +/- ##
==========================================
+ Coverage 96.09% 96.10% +0.01%
==========================================
Files 730 730
Lines 23130 23134 +4
Branches 2705 2699 -6
==========================================
+ Hits 22226 22234 +8
+ Misses 641 638 -3
+ Partials 263 262 -1 ☔ View full report in Codecov by Sentry. |
The frontend applies OR-logic when evaluating validation plugins on a Formio component, which the backend did not replicate correctly. Backport-of: #4136
Backported to 2.6.x via 5e8bb37 |
Closes #4131
I am schtewpid