Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Exclude Drag and Drop blocks from the link-scanning step in the Course Optimizer #36321

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

fix: Exclude Drag and Drop blocks from the link-scanning step in the Course Optimizer #36321

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
df5cb30
fix: ignore drag and drop in course optimizer scanning links step
hinakhadim Mar 4, 2025
f0a52a4
chore: refactor code and add test case
hinakhadim Mar 5, 2025
d0bf378
fix: lint issues
hinakhadim Mar 5, 2025
47babe2
fix: pycodecstyle issue
hinakhadim Mar 5, 2025
0dd9e1a
refactor: add self.assertFalse instead of assert
hinakhadim Mar 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions cms/djangoapps/contentstore/tasks.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1199,11 +1199,12 @@ def _scan_course_for_links(course_key):

for block in blocks:
block_id = str(block.usage_key)
block_info = get_block_info(block)
block_data = block_info['data']
if block.category != 'drag-and-drop-v2':
block_info = get_block_info(block)
block_data = block_info['data']

url_list = _get_urls(block_data)
urls_to_validate += [[block_id, url] for url in url_list]
url_list = _get_urls(block_data)
urls_to_validate += [[block_id, url] for url in url_list]

return urls_to_validate

Expand Down
Loading