Closed SoryRawyer closed 1 year ago
Thanks for the pull request, @SoryRawyer! 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:
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.
@SoryRawyer 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.
Released as v2.2.1
This change replaces the
course_key
field infact_problem_responses
with the field fromproblem_events
instead of the field fromcourse_names
. This was preventing queries from properly using thecourse_key
index onproblem_events
.This also includes a fix for the
fact_learner_problem_summary
test, which was pointing to an earlier name of that model.