feat: solve bug gradebook

[Yagnesh1998]

as per 384 number bug

Untitled.design.mp4

LMS Logs

> tutor_nightly_dev-lms-1  |     return self.get_db_prep_value(value, connection=connection, prepared=False)
> tutor_nightly_dev-lms-1  |   File "/openedx/venv/lib/python3.8/site-packages/django/db/models/fields/__init__.py", line 947, in get_db_prep_value
> tutor_nightly_dev-lms-1  |     value = self.get_prep_value(value)
> tutor_nightly_dev-lms-1  |   File "/openedx/venv/lib/python3.8/site-packages/django/db/models/fields/__init__.py", line 1959, in get_prep_value
> tutor_nightly_dev-lms-1  |     raise e.__class__(
> tutor_nightly_dev-lms-1  | ValueError: Field 'earned_graded_override' expected a number but got ''.

Solution

The || [] in the line const gradeOverrides = selectors.grades.useGradeData().gradeOverrideHistoryResults || []; provides a fallback. If gradeOverrideHistoryResults is falsy (like undefined, null, or 0), it defaults to an empty array []. This ensures gradeOverrides always has an array value, preventing potential errors.

[openedx-webhooks]

Thanks for the pull request, @Yagnesh1998! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.98%. Comparing base (d683e87) to head (e759ff1).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #388   +/-   ##
=======================================
  Coverage   94.98%   94.98%           
=======================================
  Files         139      139           
  Lines        1355     1355           
  Branches      264      265    +1     
=======================================
  Hits         1287     1287           
  Misses         60       60           
  Partials        8        8           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[farhaanbukhsh]

@mphilbrick211 @arbrandes I can help with reviewing this PR :)

Hey @Yagnesh1998 a similar PR is already in progress #387

[mphilbrick211]

@mphilbrick211 @arbrandes I can help with reviewing this PR :)

Hey @Yagnesh1998 a similar PR is already in progress #387

@Yagnesh1998 @farhaanbukhsh - can this be closed?

[Yagnesh1998]

@mphilbrick211 Yes! Both the PRs are same, there is no point in opening both the PRs, so you can definitely close it.

[openedx-webhooks]

@Yagnesh1998 Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.