-
Notifications
You must be signed in to change notification settings - Fork 7
Handle new exerciseKeys #1413
Copy link
Copy link
Labels
maintenanceFor pull requests that should not be added to the `Features, fixes, enhancements` release notesFor pull requests that should not be added to the `Features, fixes, enhancements` release notes
Metadata
Metadata
Assignees
Labels
maintenanceFor pull requests that should not be added to the `Features, fixes, enhancements` release notesFor pull requests that should not be added to the `Features, fixes, enhancements` release notes
Type
Projects
Status
In Review
Describe the Problem
In https://github.com/digitalfabrik/lunes-cms/pull/709/changes, we changed the exercise keys to strings from numbers. We need to update that in the App.
Note: While we will lose some data if there is a mismatch between the CMS and App releases here, the user won't notice it, it will be sent to Sentry if tracking is enabled. So we'll probably get some annoying errors on Sentry that we can't do anything about, but we won't be bugging the user.
Describe Your Preferred Solution
Update the code to use strings for the exerciseKeys and add a migration.
Describe Possible Alternatives
Nope.
Related Issues
Additional Information