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

Cannot edit the section order of a started quiz #12289

Closed
pcenov opened this issue Jun 17, 2024 · 2 comments · Fixed by #12307
Closed

Cannot edit the section order of a started quiz #12289

pcenov opened this issue Jun 17, 2024 · 2 comments · Fixed by #12307
Assignees
Labels
P0 - critical Priority: Release blocker or regression

Comments

@pcenov
Copy link
Member

pcenov commented Jun 17, 2024

Observed behavior

Observed while regression testing the changes made in #12278:

If I have created and started a quiz, when I go to edit it and change the section order then the quiz can no longer be saved and there is a 400 error in the console: Cannot update learners_see_fixed_order on an Exam object

cannotupdate

Logs:

logs.zip

Video:

2024-06-17_15-10-08.mp4

Expected behavior

To be further discussed whether it should be possible to update the section order of a started quiz.

Steps to reproduce the issue

  1. Install the build from Update to allow and implement randomization of sections. #12278
  2. Import the exercises from the QA channel.
  3. Create a quiz, save it and start it.
  4. Attempt to change the section order of the quiz by editing it.

Usage Details

Kolibri kolibri_0.16.2a0.dev0+git.1258.g6ca0dd28-0ubuntu1_all.deb
Windows 11, Ubuntu 22 - Chrome

@pcenov
Copy link
Member Author

pcenov commented Jun 17, 2024

@radinamatic

@rtibbles
Copy link
Member

Hrm, good point - I had disallowed this from the backend, but as it doesn't change the content of the quiz, I think this is actually safe.

And at the very least, the UI should only allow what the API allows!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 - critical Priority: Release blocker or regression
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants