Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Decouple Harvesting to Flask Admin App from pushing to CKAN #4865

Open
4 tasks
btylerburton opened this issue Aug 27, 2024 · 3 comments
Open
4 tasks

Decouple Harvesting to Flask Admin App from pushing to CKAN #4865

btylerburton opened this issue Aug 27, 2024 · 3 comments
Labels
H2.0/Harvest-Flask Front-end webapp & APIs

Comments

@btylerburton
Copy link
Contributor

User Story

In order to make the H2.0 application processes more discrete, datagovteam wants to separate the harvesting of records into the flask app from pushing the records to CKAN.

Acceptance Criteria

[ACs should be clearly demoable/verifiable whenever possible. Try specifying them using BDD.]

  • GIVEN I want to harvest a valid harvest source
    WHEN the Flask admin app has harvested the records in full and without job level errors
    THEN the harvesting app will invoke the sync to CKAN

  • GIVEN the sync to CKAN occurs independently of the harvesting process
    THEN I should be able to run the sync N number of times without having to reharvest the records to Flask Admin.

Background

[Any helpful contextual notes or links to artifacts/evidence, if needed]

Security Considerations (required)

[Any security concerns that might be implicated in the change. "None" is OK, just be explicit here!]

Sketch

  • Decouple harvesting of records to Flask Admin from the Sync to CKAN
  • Create triggers that can invoke a CKAN sync independent of a harvest invocation
@btylerburton btylerburton added the H2.0/Harvest-Flask Front-end webapp & APIs label Aug 27, 2024
@rshewitt
Copy link
Contributor

how will the sync-to-ckan process know which data to use?

@btylerburton
Copy link
Contributor Author

Do you think we'd be able to use harvest_job_id to fetch all records posted during write_compare_to_db? I believe so, but will confirm in a test.

@rshewitt
Copy link
Contributor

ah yeah

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
H2.0/Harvest-Flask Front-end webapp & APIs
Projects
Status: H2.0 Backlog
Development

No branches or pull requests

2 participants