Skip to content

Fix date validation in Excel metadata templates and add some schema validation checks#886

Merged
albatalavera merged 6 commits into
BU-ISCIII:developfrom
albatalavera:develop
May 26, 2026
Merged

Fix date validation in Excel metadata templates and add some schema validation checks#886
albatalavera merged 6 commits into
BU-ISCIII:developfrom
albatalavera:develop

Conversation

@albatalavera
Copy link
Copy Markdown
Contributor

@albatalavera albatalavera commented May 22, 2026 •
edited
Loading

  • Fixes Excel metadata template date validation so valid YYYY-MM-DD values are accepted.
  • Detects duplicated enum values during schema validation.
  • Adds validation to ensure example values are included in enum lists.

Fixes #884

PR Checklist

  • This comment contains a description of changes (with reason).
  • Make sure your code lints (black and flake8).
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

@albatalavera albatalavera linked an issue May 22, 2026 that may be closed by this pull request
Excel metadata template rejects valid YYYY-MM-DD date values #884
Closed
Daniel-VM
Daniel-VM approved these changes May 26, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@Daniel-VM Daniel-VM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@albatalavera albatalavera merged commit 3c73071 into BU-ISCIII:develop May 26, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Daniel-VM Daniel-VM Daniel-VM approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Excel metadata template rejects valid YYYY-MM-DD date values

2 participants

@albatalavera @Daniel-VM