Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Migrate injections to prepare for new Exercise Group format #260

Closed
3 tasks done
elbotho opened this issue Feb 9, 2024 · 0 comments · Fixed by #275
Closed
3 tasks done

Migrate injections to prepare for new Exercise Group format #260

elbotho opened this issue Feb 9, 2024 · 0 comments · Fixed by #275
Assignees

Comments

@elbotho
Copy link
Member

elbotho commented Feb 9, 2024

When removing GroupedExercise after #252 is done we would break > 700 injections.

To avoid this we need a migration that changes the state of all injections that target a GroupedExercise.

Instead of {grouped-exercise-uuid} or /{grouped-exercise-uuid} it should be:
{exercise-group-uuid}#{grouped-exercise-uuid} than we can restore the existing functionallity in the frontend.

Optional bonus points:

Potentially helpful:
https://github.com/serlo/db-migrations/blob/main/src/utils/edtr-io.ts

status

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

Successfully merging a pull request may close this issue.

3 participants