Skip to content

Enhances data integrity with automated cleanup#8

Merged
dpanta94 merged 15 commits into
mainfrom
feat/cleanups
Jul 14, 2025
Merged

Enhances data integrity with automated cleanup#8
dpanta94 merged 15 commits into
mainfrom
feat/cleanups

Conversation

@dpanta94

@dpanta94 dpanta94 commented Jul 11, 2025

Copy link
Copy Markdown
Member

Improves Shepherd's data integrity and efficiency by implementing automated database cleanup:

  • Introduces a "Herding" task to periodically remove orphaned task data.
  • Adds automatic cleanup of task data when Action Scheduler deletes actions, preventing orphaned records.
  • Documents database cleanup strategies and API changes.

@dpanta94 dpanta94 requested a review from lucatume July 11, 2025 14:52
@dpanta94 dpanta94 self-assigned this Jul 11, 2025

@lucatume lucatume 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.

The code looks fine, but I think testing should be improved and simplified.

Comment thread tests/integration/Provider_Test.php Outdated
Comment thread tests/integration/Provider_Test.php Outdated
Comment thread tests/wpunit/Provider_Test.php Outdated
Comment thread tests/wpunit/Regulator_Test.php Outdated
@dpanta94 dpanta94 requested review from bordoni and lucatume July 13, 2025 20:53
Comment thread docs/getting-started.md Outdated
Comment thread src/Tasks/Herding.php Outdated
Comment thread src/Tasks/Herding.php Outdated
Comment thread src/Tasks/Herding.php Outdated
@dpanta94 dpanta94 merged commit 3fcb8e0 into main Jul 14, 2025
11 checks passed
@dpanta94 dpanta94 deleted the feat/cleanups branch July 14, 2025 14:22
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.

3 participants