Skip to content

DOI Validation - #1484

Draft
JVickery-TBS wants to merge 8 commits into
feature/single-instancefrom
feature/doi-validation
Draft

DOI Validation#1484
JVickery-TBS wants to merge 8 commits into
feature/single-instancefrom
feature/doi-validation

Conversation

@JVickery-TBS

Copy link
Copy Markdown
Contributor

Adds validation to the DOI field. Also, the validators in the schema yaml was missing a tab in, so this field has never had any validators for it (very not good).

- Added validation to `digital_object_identifier` field.
- Added pytest for DOI validation.
- Added change log file.
Comment thread ckanext/canada/tests/test_validators.py Outdated
Comment thread ckanext/canada/validators.py Outdated
@JVickery-TBS
JVickery-TBS changed the base branch from master to feature/single-instance June 2, 2026 15:53
@JVickery-TBS
JVickery-TBS marked this pull request as draft July 30, 2026 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants