Form Validation: Added support for required fields #18
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
Currently the form does not support required fields. There is an array called
required
containing all the fields in the schema that can be used to determined whether a field is mandatory or not.Solution
Implemented required fields into the form by:
determineValidation
that will create an object with validation information. It includes the logic for the determining if a field is required is done here. For any form validation we add onto in the future, all functionality will be made here.validate
object is then appended to the Form.io component to be returned.Result
For required fields, the form will mark it as a required through a red asterisk. The submit button becomes enabled when all form fields are filled out.
Testing
Tested locally