Skip to content

Implement --updated option for date filtering - #599

Draft
polyrobin wants to merge 1 commit into
eprints:masterfrom
polyrobin:patch-7
Draft

Implement --updated option for date filtering#599
polyrobin wants to merge 1 commit into
eprints:masterfrom
polyrobin:patch-7

Conversation

@polyrobin

Copy link
Copy Markdown
Contributor

Added support for the --updated option to filter records modified since a specified date, with support for relative and absolute date formats.

Added support for the --updated option to filter records modified since a specified date, with support for relative and absolute date formats.
@polyrobin
polyrobin marked this pull request as draft July 27, 2026 19:05

@drn05r drn05r left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is a good idea but unless I have missed something EPrints::Update::Views does not have an update_view_by_date function. Maybe you forgot to add perl_lib/EPrints/Update/Views.pm to this pull request?

@drn05r drn05r added the enhancement New feature or request label Jul 28, 2026
@drn05r drn05r added this to the 3.4.8 milestone Jul 28, 2026
@polyrobin

Copy link
Copy Markdown
Contributor Author

yes this is still a draft, my pull request was a bit too enthousiastic. 😄 still have some more testing to do on this one since there is a lot of eprints that are related to each other and I don't want to leave anyone out. But it already cuts the processing time drastically compared to updating the whole database.

@drn05r

drn05r commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

@polyrobin I don't think I have ever used draft PRs before. I would have thought it would have: prevented reviewing, stopped you selecting approve in a PR review or at very least put a very visible banner saying "are you sure you want to review this whilst it is still a draft?".

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants