Skip to content

WIP: Added command to purge orphaned checkout acceptances #16603

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

Draft
wants to merge 14 commits into
base: develop
Choose a base branch
from

Conversation

marcusmoore
Copy link
Collaborator

This PR adds a command to purge orphaned checkout acceptances.

Copy link

what-the-diff bot commented Mar 31, 2025

PR Summary

  • Introduced a new function for system clean up
    A fresh console command named PurgeOrphanedCheckoutAcceptances was added. This function is geared towards deleting leftover data or "orphaned checkout acceptances" coming from deleted users. This puzzle piece will aid in keeping the system clean and efficient.

  • Updated Asset Acceptance Report
    The file ReportsController.php was updated to include new relationship data for both company and model category. This change is a clear improvement in the asset acceptance report as it now gives a more comprehensive perspective.

  • Optimized Testing Dependencies
    CheckoutAcceptanceFactory.php has been improved with the inclusion of new states: forAsset, forConsumable, and forLicenseSeat. These components aim to smooth out and assist the testing procedures, thereby enhancing the quality of updates in the future.

  • New test for system cleanup
    To ensure the consistent performance of newly introduced functionalities, a new test called PurgeOrphanedCheckoutAcceptancesTest was created. This test validates the functionality of the clean-up command and makes sure that all orphaned data pieces are successfully deleted, which again aids in minimizing the clutter in the system.

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

Successfully merging this pull request may close these issues.

1 participant