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

0.17.3 - Setup wizard - Cannot import a learner on a LOD from a device with multiple facilities #12872

Closed
pcenov opened this issue Nov 21, 2024 · 4 comments · Fixed by #12893
Closed
Assignees
Labels
P0 - critical Priority: Release blocker or regression

Comments

@pcenov
Copy link
Member

pcenov commented Nov 21, 2024

Observed behavior

It's not possible to import a learner on a LOD from a device with multiple facilities because at the 'Select learning facility' page there is only a 'Go back' button:

2024-11-21_13-11-24.mp4

The issue is extant in both 0.17.3 and 0.18 and is probably caused by changes made in #12397

Expected behavior

It should be possible to import a learner on a LOD from a device with multiple facilities

Steps to reproduce the issue

  1. Install Kolibri 0.17.3 and create a server device with at least 2 facilities.
  2. Install Kolibri 0.17.3 on another device.
  3. Go through the setup wizard by selecting Group learning > Learn-only device > Import one or more existing user accounts from an existing facility

Logs

LoD-logs.zip
server-logs.zip

Usage Details

Kolibri 0.17.3 and 0.18
Ubuntu 22 - Chrome

@pcenov
Copy link
Member Author

pcenov commented Nov 21, 2024

@radinamatic

@rtibbles rtibbles added the P0 - critical Priority: Release blocker or regression label Nov 21, 2024
@rtibbles rtibbles self-assigned this Nov 21, 2024
@rtibbles rtibbles added this to the upcoming patch milestone Nov 21, 2024
@radinamatic
Copy link
Member

Just to reiterate that this was discovered while testing 0.18 assets, and confirmed as replicable on 0.17 branch too, so the fix needs to be applied to both branches.

@rtibbles
Copy link
Member

Note that this bug also affects importing facilities from a device with multiple facilities.

@rtibbles
Copy link
Member

rtibbles commented Dec 3, 2024

Fixed in #12893

@rtibbles rtibbles closed this as completed Dec 3, 2024
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.

3 participants