-
Notifications
You must be signed in to change notification settings - Fork 11
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.
LGTM
Codecov Report
@@ Coverage Diff @@
## master #34 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 25 25
Lines 1294 1291 -3
Branches 194 194
=====================================
- Hits 1294 1291 -3
Continue to review full report at Codecov.
|
5e8942f
@Blackbaud-AlexKingman Ready for another look. |
@@ -1,4 +1,4 @@ | |||
export const enum SkyDateRangeCalculatorType { | |||
export enum SkyDateRangeCalculatorType { |
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.
Adding const
was causing this enum to return an empty value. I'm guessing it has something to do with our version of TypeScript. At any rate, removing the const
fixes the problem.
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.
Works great now. Thanks!
Validation works fine when the initial value is set in the
FormControl
, or if the value is written programmatically to theformControlName
after the fact, but validation doesn't work when manually selecting dates from the date pickers. This PR addresses that problem.To test, serve the
master
branch and select "Specific range" and then deliberately choose an end date that comes before a start date to NOT see the expected error.