Conversation
0923d42 to
161b144
Compare
|
I have updated the PR with the latest |
…vided error message if any, quasar component compatible for error status
…idationError added.
161b144 to
5219c24
Compare
|
I have added a fix when checking for Required error on a boolean value. You can try this PR from my project obiba/quasar-ui-amber, and go to the test page BlitzarFormItemsStepsLayoutTest.vue. |
|
Happy new year, any update about this PR? Validation errors on not visible fields is currently a blocker for me. |
|
@ymarcon happy new year
This one I agree with, I want to merge it. For the rest, in order to decide what to do I think I want to finish the Framework examples for the documentation. I want to experience the exact issue with Quasar to make an informative decision.
Do you mean |
|
Yes, sorry I meant |
While attempting to make a multi-step form (using the
showConditionto hide fields that are not part of the current step), I had the issue that "on next step" action, thevalidateFormPerSchema()call was returning errors from the hidden fields. I could have ignored errors from the fields that are not in the step (extra work), but for the form logic it is still problematic to have errors on fields that cannot be modified.=>
validateFieldPerSchema()does not perform validations on fields for whichshowConditionis false.I had also the issue that the
errorprop is expected to be a Boolean by Quasar'sQInput.=> In
validateFieldPerSchema()and inevaluateError(), whenerroris a Boolean and isfalse, the error message is looked up in theerrorMessageprop (if not found, a defaultfieldValidationErroris shown). Whenerrorprop returns a string the original behavior is still operational.