openedx / openedx/openedx-platform
Move grading signals to openedx-events?
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 8.2k
- Forks
- 4.4k
- Avg merge
- 6d 18h
- Merged PRs (30d)
- 42
Description
These ones: https://github.com/openedx/edx-platform/blob/master/lms/djangoapps/grades/signals/signals.py
Moving these to openedx-events would allow us to remove an improper (openedx->lms) import:
openedx.core.djangoapps.xblock.runtime.runtime -> lms.djangoapps.grades.api
because then the xblock runtime would depend on the openedx-event grading signals rather than the LMS grading app.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with lms/djangoapps/grades/signals/signals.py and trace the openedx.core.djangoapps.xblock.runtime.runtime to lms.djangoapps.grades.api dependency described in the issue. Determine how the grading signals should be represented in openedx-events and verify that the runtime no longer imports the LMS grading app while the signals remain available to their consumers.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100