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

Add notice for pre-0.17.x quizzes which are not started (gracefully handle edge case) #12954

Open
nucleogenesis opened this issue Dec 17, 2024 · 0 comments
Labels
APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) TODO: needs decisions Design or specifications are necessary

Comments

@nucleogenesis
Copy link
Member

nucleogenesis commented Dec 17, 2024

Overview

In 0.17 we changed the structure of Exams' question_sources field. Users may edit quizzes directly thanks to the DraftExam (editable) and Exam (non-editable) models. A user creates a new quiz in >=0.17.x and they create a DraftExam - then, when the quiz is started, it is converted to an Exam.

However, the DraftExam model did not exist prior to 0.17.x so there could be a case where a user created a quiz, not started it and then upgraded their Kolibri.

In this case, we can tell the user something like:

"This quiz was created using an older version of Kolibri and cannot be edited directly. To edit this quiz, you can make a copy of it and that copy will be editable. Sorry for the confusion."

@radinamatic and/or @carinediaz and/or @jtamiace should be consulted for the specifics for the message.

Note that this is a rather unlikely edge case


See the CreateExamPage component where the CreateQuizSection component is displayed. This is using the draft property which indicates that you're working with a DraftExam.

In this particular case, you will have draft=false AND data_model_version < 3 AND active=false -- this indicates that the quiz is not a draft, nor has it been started and it comes from before DraftExam existed.

@nucleogenesis nucleogenesis added TODO: needs decisions Design or specifications are necessary APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) labels Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) TODO: needs decisions Design or specifications are necessary
Projects
None yet
Development

No branches or pull requests

1 participant