Skip to content

Implement PERSISTENT_SUBSECTION_GRADE_CHANGED event#38862

Draft
bmtcril wants to merge 2 commits into
masterfrom
bmtcril/events_persistent_subsection_grade_changed
Draft

Implement PERSISTENT_SUBSECTION_GRADE_CHANGED event#38862
bmtcril wants to merge 2 commits into
masterfrom
bmtcril/events_persistent_subsection_grade_changed

Conversation

@bmtcril

@bmtcril bmtcril commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Description

This dusts off @mgwozdz-unicon 's #37978 for testing openedx/openedx-events#587 . It adds a new Openedx Event PERSISTENT_SUBSECTION_GRADE_CHANGED in order to support CBE's need to assess student competency at a skill.

In draft until I can replace the branch references in requirements with a new release of openedx-events.

Current test failures are all query count. Mostly going from 47 to 48, one went from 44 to 48. I'm going to take a look at that one to try and understand if there are things we can do to improve it.

Supporting information

Link to other information about the change, such as Jira issues, GitHub issues, or Discourse discussions.
Be sure to check they are publicly readable, or if not, repeat the information here.

Testing instructions

If you are set up with an event bus configuration you can test all the way through by completing a problem in a graded subsection. The lms-worker should attempt to serialize and send the event. I've tested this path locally, and it worked, though I can't guarantee it will work with every problem currently.

Tycho Hob added 2 commits July 8, 2026 16:18
Adds a new openedx event that fires when a persistenty subsection grade
changes and is event bus compatible. Needed for CBE work to help rate
student mastery.
@bmtcril bmtcril requested review from mgwozdz-unicon and ormsbee July 8, 2026 20:27
@bmtcril bmtcril changed the title Bmtcril/events persistent subsection grade changed Implement PERSISTENT_SUBSECTION_GRADE_CHANGED event Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant