-
Notifications
You must be signed in to change notification settings - Fork 243
FOUR-28600: API to delete cases and the related requests in Swagger #8675
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
base: develop
Are you sure you want to change the base?
Conversation
FOUR-28685: Create an API to delete Cases and related Requests
|
You have run out of free Bugbot PR reviews for this billing cycle. This will reset on January 21. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
|
QA server K8S was successfully deployed https://ci-5a04878444.engk8s.processmaker.net |
|
QA server K8S was successfully deployed https://ci-5a04878444.engk8s.processmaker.net |
Record deletion moved to trait to resolve sonarqube complexity issue (too many methods in DeleteCase)
|
QA server K8S was successfully deployed https://ci-5a04878444.engk8s.processmaker.net |
FOUR-28723 | Store Logs For Deleting Cases via API
FOUR-28794: Delete notifications tied to case on delete and add test
|
QA server K8S was successfully deployed https://ci-5a04878444.engk8s.processmaker.net |
FOUR-28838: Fix case detail error when case is deleted via API
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
|
|
QA server K8S was successfully deployed https://ci-5a04878444.engk8s.processmaker.net |





Issue & Reproduction Steps
Admins need an API endpoint to delete Cases and their related Requests, as described in the product document. The absence of this endpoint forces manual cleanup and can leave inconsistent data.
Solution
How to Test
Related Tickets & Packages
ci:deploy
...
Code Review Checklist
Note
Adds a transactional case deletion API and security logging.
DELETE /cases/{case_number}inCaseControllerwith OpenAPI docs and route; returns 204/404DeleteCaseaction +DeletesCaseRecordstrait to cascade-deleteprocess_requests,process_request_tokens, locks, scheduled tasks, inbox rules/logs, ABE tokens, comments, notifications (selected types), media,case_numbers,cases_started,cases_participated, and optionalellucian_ethos_sync_global_task_list; recount Saved Search counts post-commit when availableCaseDeletedevent (includesdeleted_at,case_number, title); registered inEventServiceProvider; dispatched after deletionCaseController;CasesController@showaborts 404 when no requests and safely selects parent/first request;InboxRuledeletes saved search only if class existsCaseDeleteTestcovers core/dependent record cleanup, notifications filtering, 404;SecurityLogsTestvalidatesCaseDeletedloggingCaseNumber,InboxRuleLog,ProcessRequest(withCaseNumber),ProcessRequestLock,ScheduledTask(forToken)Written by Cursor Bugbot for commit d6d9947. This will update automatically on new commits. Configure here.