Andal.LND/frontend-app-learning
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor!: clean up unused references to lms_web_url (#409)

Browse Source 
Before edx-platform version 19ba691,
only `lms_web_url` was exposed from the course
blocks API. Now that the API also exposes
`legacy_web_url`, we can stop falling back
to `lms_web_url` when `legacy_web_url` is
absent.

TNL-7796
This commit is contained in:
Kyle McCormick
2021-04-09 10:36:04 -04:00
committed by GitHub
parent e86f4a88cc
commit 88005ea5d2
3 changed files with 5 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/courseware/data/api.js
View File

@@ -38,8 +38,7 @@ export function normalizeBlocks(courseId, blocks) {
effortTime: block.effort_time,
id: block.id,
title: block.display_name,
// Fall back to `lms_web_url` until `legacy_web_url` is added to API (TNL-7796).
legacyWebUrl: block.legacy_web_url || block.lms_web_url,
legacyWebUrl: block.legacy_web_url,
unitIds: block.children || [],
};
break;
@@ -48,8 +47,7 @@ export function normalizeBlocks(courseId, blocks) {
graded: block.graded,
id: block.id,
title: block.display_name,
// Fall back to `lms_web_url` until `legacy_web_url` is added to API (TNL-7796).
legacyWebUrl: block.legacy_web_url || block.lms_web_url,
legacyWebUrl: block.legacy_web_url,
};
break;
default: