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.

Sign up for GitHub

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

Jump to bottom

Bring GetDKAN/harvest library into DKAN harvest module #4373

Open
dafeder opened this issue Dec 30, 2024 · 0 comments
Open

Bring GetDKAN/harvest library into DKAN harvest module #4373

dafeder opened this issue Dec 30, 2024 · 0 comments
Labels
Component: Harvest

Comments

@dafeder
Copy link
Member

dafeder commented Dec 30, 2024 •
edited
Loading

Related to #3821

We have no reason anymore to split Harvest between a module and an external library. Let's bring those library classes into the Harvest module and kill one more dependency.

Doing this will require:

  1. Remove HarvestHashesEntityDatabaseTable so that we are not depending on DatabaseTableInterface.
  2. Do any other refactoring necessary to eliminate dependence on GetDKAN/contracts.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component: Harvest
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dafeder