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

Unable to delete a Quiz in case where the resource(s) inside don't exist // case of disappearing and reappearing content in a Quiz #12388

Open
lauradanforth opened this issue Jul 1, 2024 · 3 comments
Labels
P1 - important Priority: High impact on UX

Comments

@lauradanforth
Copy link

This was on kolibri-demo on 0.16.2:

Observed behavior

While logged in as an admin, in the Plan tab I noticed that one of my quizzes had content in it that no longer existed in the platform, so I tried to delete the Quiz. I got this pop up:
image
But when trying to click the 'Delete' button, nothing happened.

When I refreshed the page, the Quiz was still there but I also noticed that the content had been restored:
image

Context

Tell us about your environment, including:

  • Kolibri version - 0.16.2
  • Operating system - Mac OSX
  • Browser - Chrome

@nucleogenesis nucleogenesis added the P0 - critical Priority: Release blocker or regression label Jul 2, 2024
@pcenov
Copy link
Member

pcenov commented Jul 3, 2024

Hmm I tested this locally on Kolibri 0.16.2 and deleting a quiz with or without resources is working correctly so this must be some strange edge case:

2024-07-03_17-17-57.mp4
2024-07-03_17-21-20.mp4

@nucleogenesis nucleogenesis self-assigned this Jul 3, 2024
@nucleogenesis
Copy link
Member

Alrighty just adding that I've also not been able to replicate the error noted here on develop either.

I think that this Sentry error is related which suggests that this is a bug on the front-end.

I think that this is very much an edge case and that we can lower the priority on this one.

IMO next steps here are to just look at the quiz deletion machinery again to work out if there is a way for the quiz's ID value to be nulled that we can stop.

@nucleogenesis nucleogenesis added P1 - important Priority: High impact on UX and removed P0 - critical Priority: Release blocker or regression labels Jul 3, 2024
@nucleogenesis nucleogenesis removed their assignment Jul 3, 2024
@AlexVelezLl
Copy link
Member

AlexVelezLl commented Jul 10, 2024

I managed to replicate this, and yes, it is actually a very edge case. It happened because the server responded a 502 error on GET /api/exams/exam/size/, and for that reason, the frontend detects as it lost the connection with the server and doesnt load the data of the exam and resources, so thats why it says that resource is missing. And when you click on delete, we dont have the exam id, and thats why the delete button doesnt work.

Then, after reloading the page the server is already working ok and doesnt return the 502 error and thats why we can see the resources again.

So this error si more of why the server was unavailable and the gateway returned a 502, and why the frontend didnt show the "trying to reconnect" backdrop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 - important Priority: High impact on UX
Projects
None yet
Development

No branches or pull requests

4 participants