From 1d186b34bf1da42111372452e8a4f31d97ce6c77 Mon Sep 17 00:00:00 2001 From: Simon Chen Date: Mon, 5 Apr 2021 16:54:58 -0400 Subject: [PATCH] MST-734 Fix production issue on Learner Onboarding Status Panel (#27248) * MST-734 Fix production issue on Learner Onboarding Status Panel Fix the prod issue where learning sequence service object missing the needed get_user_course_outline service API The user web API call currently returns 500 --- .../djangoapps/content/learning_sequences/services.py | 8 +++++++- requirements/constraints.txt | 3 --- requirements/edx/base.txt | 2 +- requirements/edx/development.txt | 2 +- requirements/edx/testing.txt | 2 +- 5 files changed, 10 insertions(+), 7 deletions(-) diff --git a/openedx/core/djangoapps/content/learning_sequences/services.py b/openedx/core/djangoapps/content/learning_sequences/services.py index d98a23ccc4..a43d6ddd59 100644 --- a/openedx/core/djangoapps/content/learning_sequences/services.py +++ b/openedx/core/djangoapps/content/learning_sequences/services.py @@ -3,7 +3,7 @@ Learning Sequences Runtime Service """ -from .api import get_user_course_outline_details +from .api import get_user_course_outline, get_user_course_outline_details class LearningSequencesRuntimeService: @@ -16,3 +16,9 @@ class LearningSequencesRuntimeService: Returns UserCourseOutlineDetailsData """ return get_user_course_outline_details(course_key, user, at_time) + + def get_user_course_outline(self, course_key, user, at_time): + """ + Returns UserCourseOutlineData + """ + return get_user_course_outline(course_key, user, at_time) diff --git a/requirements/constraints.txt b/requirements/constraints.txt index 99349579ab..b1010e98b3 100644 --- a/requirements/constraints.txt +++ b/requirements/constraints.txt @@ -33,9 +33,6 @@ django-storages<1.9 # for them. edx-enterprise==3.21.0 -# edx-proctoring 3.8.3 resulted in 500 error on StudentOnboardingStatusView. Temporarily pinned to 3.8.2 version -edx-proctoring==3.8.2 - # Upgrading to 2.12.0 breaks several test classes due to API changes, need to update our code accordingly factory-boy==2.8.1 diff --git a/requirements/edx/base.txt b/requirements/edx/base.txt index 9f136c0ed9..746b69b5b7 100644 --- a/requirements/edx/base.txt +++ b/requirements/edx/base.txt @@ -107,7 +107,7 @@ edx-milestones==0.3.1 # via -r requirements/edx/base.in edx-opaque-keys[django]==2.2.0 # via -r requirements/edx/paver.txt, edx-bulk-grades, edx-ccx-keys, edx-completion, edx-drf-extensions, edx-enterprise, edx-milestones, edx-organizations, edx-proctoring, edx-user-state-client, edx-when, lti-consumer-xblock, xmodule edx-organizations==6.9.0 # via -r requirements/edx/base.in edx-proctoring-proctortrack==1.0.5 # via -r requirements/edx/base.in -edx-proctoring==3.8.2 # via -r requirements/edx/base.in, edx-proctoring-proctortrack +edx-proctoring==3.8.3 # via -r requirements/edx/base.in, edx-proctoring-proctortrack edx-rbac==1.4.2 # via edx-enterprise edx-rest-api-client==5.3.0 # via -r requirements/edx/base.in, edx-enterprise, edx-proctoring edx-search==3.0.0 # via -r requirements/edx/base.in diff --git a/requirements/edx/development.txt b/requirements/edx/development.txt index 08e847e1e2..cff899e20f 100644 --- a/requirements/edx/development.txt +++ b/requirements/edx/development.txt @@ -120,7 +120,7 @@ edx-milestones==0.3.1 # via -r requirements/edx/testing.txt edx-opaque-keys[django]==2.2.0 # via -r requirements/edx/testing.txt, edx-bulk-grades, edx-ccx-keys, edx-completion, edx-drf-extensions, edx-enterprise, edx-milestones, edx-organizations, edx-proctoring, edx-user-state-client, edx-when, lti-consumer-xblock, xmodule edx-organizations==6.9.0 # via -r requirements/edx/testing.txt edx-proctoring-proctortrack==1.0.5 # via -r requirements/edx/testing.txt -edx-proctoring==3.8.2 # via -r requirements/edx/testing.txt, edx-proctoring-proctortrack +edx-proctoring==3.8.3 # via -r requirements/edx/testing.txt, edx-proctoring-proctortrack edx-rbac==1.4.2 # via -r requirements/edx/testing.txt, edx-enterprise edx-rest-api-client==5.3.0 # via -r requirements/edx/testing.txt, edx-enterprise, edx-proctoring edx-search==3.0.0 # via -r requirements/edx/testing.txt diff --git a/requirements/edx/testing.txt b/requirements/edx/testing.txt index a834186e73..bcfddecda4 100644 --- a/requirements/edx/testing.txt +++ b/requirements/edx/testing.txt @@ -117,7 +117,7 @@ edx-milestones==0.3.1 # via -r requirements/edx/base.txt edx-opaque-keys[django]==2.2.0 # via -r requirements/edx/base.txt, edx-bulk-grades, edx-ccx-keys, edx-completion, edx-drf-extensions, edx-enterprise, edx-milestones, edx-organizations, edx-proctoring, edx-user-state-client, edx-when, lti-consumer-xblock, xmodule edx-organizations==6.9.0 # via -r requirements/edx/base.txt edx-proctoring-proctortrack==1.0.5 # via -r requirements/edx/base.txt -edx-proctoring==3.8.2 # via -r requirements/edx/base.txt, edx-proctoring-proctortrack +edx-proctoring==3.8.3 # via -r requirements/edx/base.txt, edx-proctoring-proctortrack edx-rbac==1.4.2 # via -r requirements/edx/base.txt, edx-enterprise edx-rest-api-client==5.3.0 # via -r requirements/edx/base.txt, edx-enterprise, edx-proctoring edx-search==3.0.0 # via -r requirements/edx/base.txt