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

[#12048] Add dangling course delete script #13090

Conversation

NicolasCwy
Copy link
Contributor

Part of #12048

Outline of Solution
When DataMigrationForCourseEntitySql.java script is stopped abruptly, there may be dangling course entities related to the last migrated course, leaving the course in a half migrated state.

Created DeleteDanglingCourseScript.java to clean up any entities related to this partially migrated entity.

@NicolasCwy NicolasCwy requested a review from FergusMok April 24, 2024 10:04
@NicolasCwy NicolasCwy changed the base branch from master to v9-course-migration April 24, 2024 10:04
@NicolasCwy NicolasCwy self-assigned this Apr 24, 2024
@NicolasCwy NicolasCwy added s.ToReview The PR is waiting for review(s) c.Task Other non-user-facing works, e.g. refactoring, adding tests labels Apr 24, 2024
@NicolasCwy NicolasCwy merged commit 49caf5b into TEAMMATES:v9-course-migration Apr 24, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c.Task Other non-user-facing works, e.g. refactoring, adding tests s.ToReview The PR is waiting for review(s)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants