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

Improve error management and allow the user to retry and report errors during "Setting Up Kolibri" #11574

Closed
6 tasks
marcellamaki opened this issue Dec 1, 2023 · 3 comments · Fixed by #11655
Closed
6 tasks
Assignees
Labels
P0 - critical Priority: Release blocker or regression

Comments

@marcellamaki
Copy link
Member

marcellamaki commented Dec 1, 2023

Observed behavior

When the user is on the "Setting Up Kolibri" page of the setup wizard, we do not have a good error management experience.

In some instances, content syncing tasks begin running in the background once the user is imported from the facility, and this can create a lock on the database.

In other cases, errors with the import itself (such as the issue Radina reported with the "on my own flow" on Android) do not have a clear step of what to do next.

Finally, when there is an error screen on android, much of the text is cut off, making troubleshooting difficult.

Errors and logs

UI example:

Expected behavior

  • If we get an error back from the provision device endpoint, we should be able to retry, rather than the user getting stuck at the end of the setup wizard.
  • Other errors should also be better managed
  • We should enhance our middleware to catch the 503 error, and manage that case, so that the user is able to retry.
  • We should repurpose the generic "something went wrong" error message, which would allow the user to report the error, and would pop up the modal that properly displays the error text
  • We should add a retry button
  • If the retry is unsuccessful, the user should be able to start over at the beginning of the setup wizard
@nucleogenesis
Copy link
Member

@marcellamaki @rtibbles should this be in our backlog and targeted/prioritized for 0.16?

In any case I'd love to tackle this / clean up the mess I've made here and/or cohack w/ someone on it!

@marcellamaki marcellamaki added the P0 - critical Priority: Release blocker or regression label Dec 15, 2023
@pcenov
Copy link
Member

pcenov commented Jan 4, 2024

Hi @marcellamaki while testing #11655 I was able to identify the following steps to reliably replicate the error mentioned here - it's caused by leftover data from the previous installation.

Steps to replicate:

  1. Install the Android app.
  2. Go through the On my own flow.
  3. Stop the app and uninstall it without clearing any of the app data or cache.
  4. Install again the app and go through the On my own flow.

@marcellamaki
Copy link
Member Author

Thank you @pcenov!

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.

4 participants