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

Submission status job filters by submission date #98

Merged
merged 2 commits into from
Mar 6, 2024

Conversation

markpatton
Copy link
Contributor

@markpatton markpatton commented Mar 6, 2024

The submission status job now like the deposit status job only updates submissions within a certain window of days of submitted date. They both use the same property.

Changes:

  • Rename property pass.deposit.update.window.days to pass.status.update.window.days.
  • Change submission status job to only check submissions within pass.status.update.window.days days.
  • Minor cleanups

….window.days. Change submission status job to only check submissions within pass.status.update.window.days days.
Copy link
Contributor

@rpoet-jh rpoet-jh left a comment

Choose a reason for hiding this comment

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

Good job, Mark. Just one minor comment about a test.

@markpatton markpatton requested a review from rpoet-jh March 6, 2024 16:07
@markpatton markpatton merged commit 844bd85 into main Mar 6, 2024
2 checks passed
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.

Add Submission.submittedDate filter to deposit-services submission status job
2 participants