-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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(datepicker): expose validators minDate, maxDate, invalidDate #3126
feat(datepicker): expose validators minDate, maxDate, invalidDate #3126
Conversation
Hey @maxokorokov. Is there any chance for getting this reviewed? If you have any remarks, l would be happy to apply them. |
5d7ba8c
to
b56b8e7
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.
Thanks @03byron!
I've just rebased on master because of conflicts and added comment for the NgbDateValidators
:
Cheers,
Max
Codecov Report
@@ Coverage Diff @@
## master #3126 +/- ##
==========================================
+ Coverage 92.04% 92.07% +0.02%
==========================================
Files 91 92 +1
Lines 3054 3090 +36
Branches 505 513 +8
==========================================
+ Hits 2811 2845 +34
Misses 179 179
- Partials 64 66 +2
Continue to review full report at Codecov.
|
…Date (ng-bootstrap#3126)" This reverts commit 509bba3.
…Date (ng-bootstrap#3126)" (ng-bootstrap#3219) This reverts commit 509bba3.
Follow up for #3115.