learningequality / kolibri

Kolibri Learning Platform: the offline app for universal education
https://learningequality.org/kolibri/
MIT License
814 stars 687 forks source link

Learner redirected to 'Select/Change facility' again after signing in with the new credentials after the 'Merge account' #12762

Open radinamatic opened 4 weeks ago

radinamatic commented 4 weeks ago

Observed behavior

Discovered while testing https://github.com/learningequality/kolibri/pull/12761.

@rtibbles suggested that the error might be generated when trying to access the "login in as merged user" endpoint - probably a 403 error...

Errors and logs

Expected behavior

More seamless experience after the merge: if the authentication is necessary, user should land on the Learn home page.

User-facing consequences

Confusion, was the merge completed as requested, or not...?

Steps to reproduce

(1:48 in the video below)

https://github.com/user-attachments/assets/001d7622-ec88-4602-a209-fe0760842044

Context

emily-dominguez commented 2 weeks ago

hi! could i get this assigned?

rtibbles commented 2 weeks ago

Yes, @emily-dominguez - do note that effectively testing this workflow will involve running multiple Kolibri servers at once, in order to have a server to connect to, in order to merge your 'on my own setup' user into.

Please feel free to ask any questions here as you are exploring the issue.

emily-dominguez commented 2 weeks ago

i see, or actually is help still wanted here (https://github.com/learningequality/kolibri/issues/12647) ?

rtibbles commented 2 weeks ago

I commented on that issue - I don't think it's suitable for contribution at this juncture.

rtibbles commented 2 weeks ago

If this issue seems too much, I have made sure to clear assignees from other help wanted issues where there had not been contribution in a long time. Please see the help wanted issues here: https://github.com/learningequality/kolibri/issues?q=is%3Aissue%20state%3Aopen%20label%3A%22help%20wanted%22

emily-dominguez commented 52 minutes ago

@rtibbles if this may be too much, what would be the best issues to start off with?

rtibbles commented 14 minutes ago

@emily-dominguez see the link I put above for a list of help wanted issues.