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(forms): Add float number support for min and max validator #42223
fix(forms): Add float number support for min and max validator #42223
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.
@iRealNirmal thanks for the fix 👍
The code looks good, I've just left a comment about extra tests that we'd need to add.
f2968ba
to
362b022
Compare
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.
One small suggestion.
362b022
to
e20c0ce
Compare
e20c0ce
to
55dc160
Compare
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.
@iRealNirmal thanks for adding tests and updating the code 👍
I've just left one final comment and we should be ready for final checks in Google's codebase and adding this PR to the merge queue if everything goes well.
Added float number support for minValidator and maxValidator Closes angular#42215
55dc160
to
85a21b3
Compare
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 great, thanks @iRealNirmal 👍
I've started tests in Goolge's codebase (internal-only link) and will keep this thread updated.
@iRealNirmal FYI presubmits in Google's codebase went well, I'm adding this PR to the merge queue. |
…ar#42223) Added float number support for minValidator and maxValidator Closes angular#42215 PR Close angular#42223
…ar#42223) Added float number support for minValidator and maxValidator Closes angular#42215 PR Close angular#42223
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Added float number support for min and max validator
Closes #42215
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Currently min and max validators are not supporting float number
Issue Number: #42215
What is the new behavior?
Added float number support for min and max validator
Does this PR introduce a breaking change?
Other information