Skip to content

Commit

Permalink
fix: remove learner_downloadable field references from libraries (ope…
Browse files Browse the repository at this point in the history
…nedx#35788)

Also bumps openedx-learning to v0.17.0, which no longer has this field.
  • Loading branch information
Ian2012 authored Nov 8, 2024
1 parent 31d170a commit d197077
Show file tree
Hide file tree
Showing 7 changed files with 5 additions and 8 deletions.
2 changes: 0 additions & 2 deletions openedx/core/djangoapps/content_libraries/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -1014,7 +1014,6 @@ def import_staged_content_from_user_clipboard(library_key: LibraryLocatorV2, use
component_version.pk,
content.id,
key=filename,
learner_downloadable=True,
)

# Emit library block created event
Expand Down Expand Up @@ -1084,7 +1083,6 @@ def _create_component_for_block(content_lib, usage_key, user_id=None):
component_version.pk,
content.id,
key="block.xml",
learner_downloadable=False
)

return component_version
Expand Down
1 change: 0 additions & 1 deletion openedx/core/djangoapps/content_libraries/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -1199,7 +1199,6 @@ def get_component_version_asset(request, component_version_uuid, asset_path):
component_version_uuid,
asset_path,
public=False,
learner_downloadable_only=False,
)

# If there was any error, we return that response because it will have the
Expand Down
2 changes: 1 addition & 1 deletion requirements/constraints.txt
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ optimizely-sdk<5.0
# Date: 2023-09-18
# pinning this version to avoid updates while the library is being developed
# Issue for unpinning: https://github.com/openedx/edx-platform/issues/35269
openedx-learning==0.16.3
openedx-learning==0.17.0

# Date: 2023-11-29
# Open AI version 1.0.0 dropped support for openai.ChatCompletion which is currently in use in enterprise.
Expand Down
2 changes: 1 addition & 1 deletion requirements/edx/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -827,7 +827,7 @@ openedx-filters==1.11.0
# -r requirements/edx/kernel.in
# lti-consumer-xblock
# ora2
openedx-learning==0.16.3
openedx-learning==0.17.0
# via
# -c requirements/edx/../constraints.txt
# -r requirements/edx/kernel.in
Expand Down
2 changes: 1 addition & 1 deletion requirements/edx/development.txt
Original file line number Diff line number Diff line change
Expand Up @@ -1381,7 +1381,7 @@ openedx-filters==1.11.0
# -r requirements/edx/testing.txt
# lti-consumer-xblock
# ora2
openedx-learning==0.16.3
openedx-learning==0.17.0
# via
# -c requirements/edx/../constraints.txt
# -r requirements/edx/doc.txt
Expand Down
2 changes: 1 addition & 1 deletion requirements/edx/doc.txt
Original file line number Diff line number Diff line change
Expand Up @@ -990,7 +990,7 @@ openedx-filters==1.11.0
# -r requirements/edx/base.txt
# lti-consumer-xblock
# ora2
openedx-learning==0.16.3
openedx-learning==0.17.0
# via
# -c requirements/edx/../constraints.txt
# -r requirements/edx/base.txt
Expand Down
2 changes: 1 addition & 1 deletion requirements/edx/testing.txt
Original file line number Diff line number Diff line change
Expand Up @@ -1041,7 +1041,7 @@ openedx-filters==1.11.0
# -r requirements/edx/base.txt
# lti-consumer-xblock
# ora2
openedx-learning==0.16.3
openedx-learning==0.17.0
# via
# -c requirements/edx/../constraints.txt
# -r requirements/edx/base.txt
Expand Down

0 comments on commit d197077

Please sign in to comment.