Skip to content

2397 check that any values in columns with a date datatype contain valid dates#2412

Closed
gibahjoe wants to merge 35 commits intomainfrom
2397-check-that-any-values-in-columns-with-a-date-datatype-contain-valid-dates
Closed

2397 check that any values in columns with a date datatype contain valid dates#2412
gibahjoe wants to merge 35 commits intomainfrom
2397-check-that-any-values-in-columns-with-a-date-datatype-contain-valid-dates

Conversation

@gibahjoe
Copy link
Copy Markdown
Contributor

Validation coverage

  • datetime (also used for date values)
  • integer
  • decimal
  • flag
  • json
  • curie
  • curie-list (override-based for organisations)
  • url
  • hash
  • pattern
  • latitude
  • longitude
  • point
  • multipolygon
  • string
  • text
  • blob
  • wkt

Notes

  • Datatypes come from field.csv in specification repo, with field-level overrides where needed.
  • Validation is inspired from specification repo especially for dateformat.

gibahjoe and others added 9 commits March 19, 2026 15:46
…pipeline-csvs' into 2396-check-that-all-entity-numbers-in-lookupcsv-are-contained-within-the-ranges-in-entity-organisationcsv
removed wrong column

made error clearer

clearer error
…csv-are-contained-within-the-ranges-in-entity-organisationcsv
…hanged the entity ranges to read from specification
…csv-are-contained-within-the-ranges-in-entity-organisationcsv

# Conflicts:
#	collection/brownfield-land/endpoint.csv
#	collection/brownfield-land/source.csv
Swati-Dash and others added 7 commits March 26, 2026 11:30
Updated heritage-at-risk entity ranges in the CSV.
Removed duplicate header and fixed buffer zone range.
…csv-are-contained-within-the-ranges-in-entity-organisationcsv

# Conflicts:
#	pipeline/local-plan/entity-organisation.csv
#	pipeline/tree-preservation-order/entity-organisation.csv
…tained-within-the-ranges-in-entity-organisationcsv' into 2397-check-that-any-values-in-columns-with-a-date-datatype-contain-valid-dates

# Conflicts:
#	collection/article-4-direction/source.csv
@gibahjoe
Copy link
Copy Markdown
Contributor Author

Closed in favour of #2396

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants