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

Bulk unpublish/archive harvested datasets #3737

Merged
merged 16 commits into from
Jan 21, 2022
Merged

Bulk unpublish/archive harvested datasets #3737

merged 16 commits into from
Jan 21, 2022

Conversation

dafeder
Copy link
Member

@dafeder dafeder commented Jan 13, 2022

Adds a new drush command, dkan:harvest:archive. Provided a harvest ID, it will archive all datasets connected to that harvest.

Note that connected items, like distributions, will not be touched. A separate PR will sync distributions workflow state to their parent datasets.

There is also a dkan:harvest:publish command that will do the inverse.

QA Steps

This harvest from demo.getdkan.com, so don't run these exact steps if you have the QA content.

Note

I don't think there's a way to keep the method count under 20 on that one file CodeClimate is complaining about. I will be trying to break a lot of this stuff up in the coming weeks to refactor the metastore into some more manageable classes; lets revisit after that?

@dafeder dafeder marked this pull request as ready for review January 20, 2022 21:46
@dafeder
Copy link
Member Author

dafeder commented Jan 21, 2022

Added a publish command. I think the redundant code complaint from codeclimate is ok for now; command classes are kind of their own thing (should clean up at some point through).

@dafeder dafeder requested a review from janette January 21, 2022 17:25
@janette janette merged commit d206040 into 2.x Jan 21, 2022
@janette
Copy link
Member

janette commented Jan 21, 2022

Keep this branch until the next tag is cut, 2.13.2

@janette janette deleted the unpub-harvest branch May 19, 2023 14:25
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.

None yet

2 participants