-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat(form-val): Adding check for gaps and laps in for field arrays #736
Conversation
Coverage Report
File Coverage
|
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.
Looks good from what I can tell. I hope it gets through QA fine though
Code Climate has analyzed commit 2f1158b and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 0.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 54.6% (-0.2% change). View more on Code Climate. |
🎉 This PR is included in version 1.5.0-val.79 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Purpose
Added a custom validation case to validate the from/to age field arrays specifically is form CS3. After some clarification from CDH it does not appear that this needs to cover the entire are range from 0-19 just that it can have no gaps, no overlaps, and from cannot be greater than the To age.
We also have an issue where we can’t compare values between inputs that live in fieldArrays. We need to add logic that regardless of which fieldArray a user is in, they can compare it with another input
Also fixed some weird padding issues for inputs with icons
Linked Issues to Close
https://qmacbis.atlassian.net/browse/OY2-29634