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

Change learning facility - Getting a 400 error "Invalid_username" while changing the facility #12191

Closed
pcenov opened this issue May 22, 2024 · 4 comments
Assignees
Labels
APP: Facility Re: Facility App (user/class management, facility settings, csv import/export, etc.) APP: User Re: User app (sign-in, sign-up, user profile, etc.) bug Behavior is wrong or broken P0 - critical Priority: Release blocker or regression

Comments

@pcenov
Copy link
Member

pcenov commented May 22, 2024

Observed behavior

When the Require password for learners option at Facility > Settings is left unchecked and I attempt to change my facility from an 'On my own' device, then I am getting into an infinite loop:

bothdevices.mp4

I checked the 0.16.1 version of Kolibri and it's extant there as well.

Expected behavior

It should be possible to change the learning facility.

Steps to reproduce the issue

  1. Install Kolibri 0.16.2b2 on two devices one of which is of type 'Formal facility' and the other one is an 'On my own'
  2. On the 'On my own' device go to the Profile page and click the 'Change' button
  3. Keep clicking the 'Continue' button and observe that clicking 'Continue' at 'Confirm account username' brings the user back to 'Change learning facility'.

##Logs

on-my-own-logs.zip

server-logs.zip

Usage Details

Kolibri 0.16.2b2
Windows 11, Ubuntu 22 - Chrome, Firefox

@pcenov
Copy link
Member Author

pcenov commented May 22, 2024

@radinamatic

@radinamatic radinamatic added bug Behavior is wrong or broken APP: User Re: User app (sign-in, sign-up, user profile, etc.) APP: Facility Re: Facility App (user/class management, facility settings, csv import/export, etc.) P0 - critical Priority: Release blocker or regression labels May 22, 2024
@radinamatic
Copy link
Member

@marcellamaki This is not a regression but looks serious... 🤔

@marcellamaki
Copy link
Member

yes, we will fix this for the patch. Thank you @pcenov !

@rtibbles
Copy link
Member

Fixed in #12193

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Facility Re: Facility App (user/class management, facility settings, csv import/export, etc.) APP: User Re: User app (sign-in, sign-up, user profile, etc.) bug Behavior is wrong or broken P0 - critical Priority: Release blocker or regression
Projects
None yet
Development

No branches or pull requests

4 participants