Skip to content

Allow zero month dates in CSV validator - #2403

Open
danloveg wants to merge 6 commits into
artefactual:qa/2.xfrom
danloveg:dev/issue-2402-validate-dates-with-zeroes
Open

Allow zero month dates in CSV validator#2403
danloveg wants to merge 6 commits into
artefactual:qa/2.xfrom
danloveg:dev/issue-2402-validate-dates-with-zeroes

Conversation

@danloveg

@danloveg danloveg commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Closes #2402

Add YYYY-00-00, YYYY-00, and YYYY-MM-00 as valid patterns to the event date validator.

@sarah-mason sarah-mason added the Community Pull requests that have been contributed from community members outside Artefactual label Aug 6, 2026
@danloveg
danloveg force-pushed the dev/issue-2402-validate-dates-with-zeroes branch from 8ce7973 to 7d9ba21 Compare August 7, 2026 20:25
@anvit

anvit commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Hi @danloveg , thanks so much for this fix! This was unfortunately not captured in the original issue, but AtoM also treats YYYY-MM-00 as valid. I think the change looks good but would you be able to update the regex pattern to also account for this format?

Thanks again!

@danloveg

Copy link
Copy Markdown
Contributor Author

Hi @danloveg , thanks so much for this fix! This was unfortunately not captured in the original issue, but AtoM also treats YYYY-MM-00 as valid. I think the change looks good but would you be able to update the regex pattern to also account for this format?

Thanks again!

No problem, I've added YYYY-MM-00 as a valid pattern to these changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Community Pull requests that have been contributed from community members outside Artefactual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Problem: CSV Validation fails when event dates have 00 for month or date.

3 participants