kolibri icon indicating copy to clipboard operation
kolibri copied to clipboard

Copying quizzes and lessons on devices with multiple facilities (superadmin role)

Open radinamatic opened this issue 3 years ago • 2 comments

Observed behavior

Uncovered while testing #8212:


When the superadmin on a device with more than one facility accesses the coach dashboard to try to copy a lesson or a quiz, they will encounter some inconsistencies. In my case the device has 2 facilities:

  • Home Facility dd, created during the initial setup wizard for the device
  • ZukVillage facility, imported from the Kolibri beta server

When in the ZukVillage facility coach dashboard I try to make a copy of a quiz or a lesson, in the step where one needs to assign it to a class, Kolibri pulls out a list of classes from the Home Facility dd instead. There seems to be no option available to select any of the other 4 classes in the facility ZukVillage, which is confusing and as it should be the default. Looks like Kolibri takes Home Facility dd as a default facility, and ONLY allows copying quizzes and lessons to it, and not into classes of the other facility imported afterwards.

ZukVillage Home Facility dd
LEDev2104 (start)  Running  - Oracle VM VirtualBox_011 LEDev2104 (start)  Running  - Oracle VM VirtualBox_012

Not sure what would happen when the device had 2 or more imported facilities and none created through the setup wizard.

Errors and logs

Expected behavior

All activity within a facility should be scoped within that facility.

  • The first step of the fix should be to make sure that by default superadmin is able to copy a quiz/lesson to classes of the facility whose coach dashboard they are currently using.
  • The second step that we may optionally choose to implement is to allow superadmins to copy across facilities, which will require some UX decision. cc @jtamiace @khangmach

User-facing consequences

Confusing and inconsistent copying workflow for superadmins in the case of more than one facility on the device.

Steps to reproduce

Context

radinamatic avatar Jul 19 '21 17:07 radinamatic

@radinamatic this one is still valid in 0.15.1 and is exactly as described - shows only the classes for the default facility.

pcenov avatar Mar 22 '22 12:03 pcenov

Thank you @pcenov.

@marcellamaki This one looks important to me, so giving it a P1 label.

radinamatic avatar Mar 22 '22 16:03 radinamatic