Skip to content

Commit

Permalink
feat: Adds disable_progress_graph attribute to the returned course_me…
Browse files Browse the repository at this point in the history
…tadata response

Currently, openedx/frontend-app-authoring#517 faces an issue when the
progress graph toggle is enabled/disabled but the settings are not respected, the disable_progress_graph
attribute will allow the frontend-app-learning repo to use this attribute to respect the settings authored
from frontend-app-course-authoring and ultimately fix openedx/frontend-app-authoring#517.

(cherry picked from commit b14396b)
  • Loading branch information
RafayGhafoor authored and Agrendalath committed Oct 15, 2024
1 parent 0cd6af4 commit 0cd497d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions lms/djangoapps/course_home_api/progress/serializers.py
Original file line number Diff line number Diff line change
Expand Up @@ -145,3 +145,4 @@ class ProgressTabSerializer(VerifiedModeSerializer):
username = serializers.CharField()
user_has_passing_grade = serializers.BooleanField()
verification_data = VerificationDataSerializer()
disable_progress_graph = serializers.BooleanField()
2 changes: 2 additions & 0 deletions lms/djangoapps/course_home_api/progress/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -228,6 +228,7 @@ def get(self, request, *args, **kwargs):

descriptor = modulestore().get_course(course_key)
grading_policy = descriptor.grading_policy
disable_progress_graph = descriptor.disable_progress_graph
verification_status = IDVerificationService.user_status(student)
verification_link = None
if verification_status['status'] is None or verification_status['status'] == 'expired':
Expand Down Expand Up @@ -257,6 +258,7 @@ def get(self, request, *args, **kwargs):
'username': username,
'user_has_passing_grade': user_has_passing_grade,
'verification_data': verification_data,
'disable_progress_graph': disable_progress_graph,
}
context = self.get_serializer_context()
context['staff_access'] = is_staff
Expand Down

0 comments on commit 0cd497d

Please sign in to comment.