-
Notifications
You must be signed in to change notification settings - Fork 347
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
Add validation for log update sliders #3389
Conversation
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
test approved |
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.
@Ashesh3 It seems we are repeating the same 2 functions in multiple files, so why don't you put them in a common file like utils.res
or helpers.res
and import the functions and use them?
and also instead of the names isInvalidInputInt
and isInvalidInputFloat
why not name it like isInputInRangeInt
and isInputInRangeFloat
, its upto you
@khavinshankar Thank you for the suggestions, I have implemented them. |
Kudos, SonarCloud Quality Gate passed!
|
Fixes #2121
Adds validation to all the sliders of all sections.