Skip to content

Commit 48d2ed9

Browse files
authored
Add a configurable suffix to Open edX course URLs (#3121)
1 parent 56d4033 commit 48d2ed9

File tree

3 files changed

+8
-2
lines changed

3 files changed

+8
-2
lines changed

b2b/api.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -291,7 +291,8 @@ def create_contract_run(
291291
live=True,
292292
b2b_contract=contract,
293293
courseware_url_path=urljoin(
294-
settings.OPENEDX_COURSE_BASE_URL, quote(new_readable_id)
294+
settings.OPENEDX_COURSE_BASE_URL,
295+
f"{quote(new_readable_id)}{settings.OPENEDX_COURSE_BASE_URL_SUFFIX}",
295296
),
296297
)
297298
course_run.save()

courses/api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1247,7 +1247,7 @@ def import_courserun_from_edx( # noqa: C901, PLR0913
12471247
is_source_run=is_source_run,
12481248
courseware_url_path=urljoin(
12491249
settings.OPENEDX_COURSE_BASE_URL,
1250-
f"/{edx_course_run.course_id}/course",
1250+
f"/{edx_course_run.course_id}{settings.OPENEDX_COURSE_BASE_URL_SUFFIX}",
12511251
),
12521252
)
12531253

main/settings.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1170,6 +1170,11 @@
11701170
default="http://edx.odl.local:18000/learn/course/",
11711171
description="The base URL to use to construct URLs to a course",
11721172
)
1173+
OPENEDX_COURSE_BASE_URL_SUFFIX = get_string(
1174+
name="OPENEDX_COURSE_BASE_URL_SUFFIX",
1175+
default="/home",
1176+
description="The suffix (with leading slash) to append to a course URL.",
1177+
)
11731178

11741179
OPENEDX_BASE_REDIRECT_URL = get_string(
11751180
name="OPENEDX_BASE_REDIRECT_URL",

0 commit comments

Comments
 (0)