Skip to content

Commit

Permalink
refactor: rename xblock render filter
Browse files Browse the repository at this point in the history
  • Loading branch information
nsprenkle committed Jun 3, 2024
1 parent f887540 commit 5429692
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions openedx_filters/learning/filters.py
Original file line number Diff line number Diff line change
Expand Up @@ -546,12 +546,12 @@ def run_filter(cls, enrollments):
return data.get("enrollments")


class RenderXBlockStarted(OpenEdxPublicFilter):
class RenderXBlockCompleted(OpenEdxPublicFilter):
"""
Filter in between context generation and rendering of XBlock scope.
"""

filter_type = "org.openedx.learning.xblock.render.started.v1"
filter_type = "org.openedx.learning.xblock.render.completed.v1"

class PreventXBlockBlockRender(OpenEdxFilterException):
"""
Expand All @@ -566,7 +566,6 @@ def run_filter(cls, context, student_view_context):
Arguments:
context (dict): rendering context values like is_mobile_app, show_title..etc.,
student_view_context (dict): context passed to the student_view, includes query params
request (request): requests object
"""
data = super().run_pipeline(context=context, student_view_context=student_view_context)
return data.get("context"), data.get("student_view_context")

Check warning on line 571 in openedx_filters/learning/filters.py

View check run for this annotation

Codecov / codecov/patch

openedx_filters/learning/filters.py#L570-L571

Added lines #L570 - L571 were not covered by tests
Expand Down

0 comments on commit 5429692

Please sign in to comment.