Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: extra validation for dates #910

Merged
merged 2 commits into from
Oct 15, 2018
Merged

fix: extra validation for dates #910

merged 2 commits into from
Oct 15, 2018

Commits on Oct 14, 2018

  1. fix: extra validation for dates 

    This commit adds an additional test for dates
over and above passing the pattern match.

It checks to assertain that the date is a
valid date. For examples, dates like
2009-02-29 are caught as invalid.

Additionally, it also fixes a minor bug
with the `weekDate` that was wrongfully
leaving out W53 as an invalid date.

fixes validatorjs#772
    @profnandaa
    profnandaa committed Oct 14, 2018
    Configuration menu
    Copy the full SHA
    e4bb296 View commit details
    Browse the repository at this point in the history

Commits on Oct 15, 2018

  1. fix: add extra validation for ordinal dates 

    - adds provision for ordinal dates
- checks that ordinal dates are also valid for leap years
- includes regression tests for previous cases with `strict = false`
    @profnandaa
    profnandaa committed Oct 15, 2018
    Configuration menu
    Copy the full SHA
    f3bb0cd View commit details
    Browse the repository at this point in the history