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

radinamatic commented 3 years ago

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

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

…

pcenov commented 2 years ago

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

radinamatic commented 2 years ago

Thank you @pcenov.

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

learningequality / kolibri