diff --git a/src/course-home/data/api.js b/src/course-home/data/api.js index afcae8ed..1e7104d8 100644 --- a/src/course-home/data/api.js +++ b/src/course-home/data/api.js @@ -56,13 +56,12 @@ export function normalizeOutlineBlocks(courseId, blocks) { effortTime: block.effort_time, icon: block.icon, id: block.id, - // 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, - // The presence of an LMS URL for the sequence indicates that we want this + legacyWebUrl: block.legacy_web_url, + // The presence of an legacy URL for the sequence indicates that we want this // sequence to be a clickable link in the outline (even though, if the new // courseware experience is active, we will ignore `legacyWebUrl` and build a // link to the MFE ourselves). - showLink: !!(block.legacy_web_url || block.lms_web_url), + showLink: !!block.legacy_web_url, title: block.display_name, }; break; diff --git a/src/courseware/data/api.js b/src/courseware/data/api.js index 72c36b04..c9fdc619 100644 --- a/src/courseware/data/api.js +++ b/src/courseware/data/api.js @@ -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: diff --git a/src/shared/data/__factories__/block.factory.js b/src/shared/data/__factories__/block.factory.js index 83fc96e6..f037c022 100644 --- a/src/shared/data/__factories__/block.factory.js +++ b/src/shared/data/__factories__/block.factory.js @@ -46,17 +46,6 @@ Factory.define('block') return `${host}/xblock/${id}`; }, ) - .attr( - 'lms_web_url', - ['legacy_web_url', 'host', 'courseId', 'id'], - (url, host, courseId, id) => { - if (url) { - return url; - } - - return `${host}/courses/${courseId}/jump_to/${id}`; - }, - ) .attr( 'legacy_web_url', ['legacy_web_url', 'host', 'courseId', 'id'],