Skip to content

Conversation

@FuhuXia
Copy link
Member

@FuhuXia FuhuXia commented Nov 18, 2025

Pull Request

Related to

About

  • Sync to dataset table after records are harvested
  • use command flask dataset check --apply to backfill existing records to dataset table
  • remove trace of CKAN code
  • add translated spatial to dataset table

PR TASKS

  • Code well documented
  • Tests written, run and passed
  • Files linted

@FuhuXia FuhuXia marked this pull request as ready for review November 25, 2025 18:13
neilmb
neilmb previously approved these changes Nov 25, 2025
Copy link
Contributor

@neilmb neilmb left a comment

Choose a reason for hiding this comment

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

This is great, so glad to see the CKAN leaving our system. Nothing big at all, comments about docstrings, etc.

Co-authored-by: Neil MartinsenBurrell <[email protected]>
Copy link
Member

@cmhedrickREI cmhedrickREI left a comment

Choose a reason for hiding this comment

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

LGTM

@FuhuXia FuhuXia merged commit d55bdb9 into main Dec 1, 2025
16 checks passed
@FuhuXia FuhuXia deleted the sync-dataset branch December 1, 2025 10:22
@FuhuXia FuhuXia mentioned this pull request Dec 1, 2025
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.

4 participants