mitodl / edx-platform

The Open edX platform, the software that powers edX!
http://open.edx.org/
GNU Affero General Public License v3.0
6 stars 1 forks source link

Data download fails for a course with malformed OLX data #262

Open pdpinch opened 3 years ago

pdpinch commented 3 years ago

Steps to Reproduce

  1. Go to https://lms.mitx.mit.edu/courses/course-v1:MITx+8.02+2021_Spring/instructor#view-data_download
  2. Click "Select a Section or Problem"
  3. Choose > Week 7 > Week 7 Sequence 1 > [3rd] Survey (you do not need to choose "Classes and Instructors" but you can. The result is the same)

Actual Result

Error: There was an error generating your report.

Expected Result

A downloadable CSV is created

Sentry error with stack trace

https://sentry.io/organizations/mit-office-of-digital-learning/issues/2328664184/?project=1757731&query=is%3Aunresolved

pdpinch commented 3 years ago

We're not sure how usable the captured date was, but if we could catch the assert error and do something else that allows the report to continue, we might be able to get more data into the report.

HamzaIbnFarooq commented 3 years ago

FYI: The issue can be reproduced by creating some inconsistency in the question problem. You can follow the below steps to reproduce the issue: 1) Create a checkboxes problem 2) Add N options under that problem (where N is a number) 3) Go to LMS, select and submit Nth option 4) Go back to the studio and update the question by removing the Nth option (now the total options will be N-1) 5) Try generating a response report for that problem, it will give Error: There was an error generating your report. error.

HamzaIbnFarooq commented 3 years ago

@pdpinch can we close this?

pdpinch commented 3 years ago

I’d like to see it deployed to lms.mitx.mit.edu first and confirm that it fixes the problem with the course.

HamzaIbnFarooq commented 3 years ago

Oh, sorry, I thought it's deployed. Please keep me updated as I have to update its related upstream PR (https://github.com/edx/edx-platform/pull/27858)

HamzaIbnFarooq commented 3 years ago

FYI: Its upstream PR (https://github.com/edx/edx-platform/pull/27858) has been merged.

shaidar commented 2 years ago

@pdpinch Can this be closed now?