Skip to content

Commit 4a75852

Browse files
arbrandesclaude
andcommitted
fix: prepend lmsBaseUrl to relative resumeUrl in ResumeButton
Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 3c2ccdc commit 4a75852

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

src/containers/CourseCard/components/CourseCardActions/ResumeButton.jsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import { EXECUTIVE_EDUCATION_COURSE_MODES } from '@src/data/constants/course';
77
import track from '@src/tracking';
88
import { useCourseTrackingEvent, useCourseData } from '@src/hooks';
99
import { useInitializeLearnerHome } from '@src/data/hooks';
10+
import { baseAppUrl } from '@src/data/services/lms/urls';
1011
import useActionDisabledState from '../hooks';
1112
import ActionButton from './ActionButton';
1213
import messages from './messages';
@@ -15,7 +16,7 @@ export const ResumeButton = ({ cardId }) => {
1516
const { formatMessage } = useIntl();
1617
const { data: learnerData } = useInitializeLearnerHome();
1718
const courseData = useCourseData(cardId);
18-
const resumeUrl = courseData?.courseRun?.resumeUrl;
19+
const resumeUrl = baseAppUrl(courseData?.courseRun?.resumeUrl);
1920
const execEdTrackingParam = useMemo(() => {
2021
const isExecEd2UCourse = EXECUTIVE_EDUCATION_COURSE_MODES.includes(courseData.enrollment.mode);
2122
const { authOrgId } = learnerData.enterpriseDashboard || {};

src/containers/CourseCard/components/CourseCardActions/ResumeButton.test.jsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ jest.mock('@src/data/hooks', () => ({
2121
jest.mock('@src/hooks', () => ({
2222
useCourseData: jest.fn().mockReturnValue({
2323
enrollment: { mode: 'executive-education' },
24-
courseRun: { homeUrl: 'home-url' },
24+
courseRun: { homeUrl: 'http://localhost:8000/courses/course-v1:Test+101+2024/course/' },
2525
}),
2626
useCourseTrackingEvent: jest.fn().mockReturnValue({
2727
trackCourseEvent: jest.fn(),
@@ -40,7 +40,7 @@ jest.mock('./ActionButton/hooks', () => jest.fn(() => false));
4040

4141
useCourseData.mockReturnValue({
4242
enrollment: { mode: 'executive-education' },
43-
courseRun: { resumeUrl: 'home-url' },
43+
courseRun: { resumeUrl: '/courses/course-v1:Test+101+2024/courseware/chapter1/' },
4444
});
4545

4646
describe('ResumeButton', () => {
@@ -84,7 +84,7 @@ describe('ResumeButton', () => {
8484
expect(useCourseTrackingEvent).toHaveBeenCalledWith(
8585
track.course.enterCourseClicked,
8686
props.cardId,
87-
`home-url?org_id=${authOrgId}`,
87+
`http://localhost:8000/courses/course-v1:Test+101+2024/courseware/chapter1/?org_id=${authOrgId}`,
8888
);
8989
});
9090
});

0 commit comments

Comments
 (0)