Skip to content

Commit

Permalink
fix: fix marketing url for programs
Browse files Browse the repository at this point in the history
  • Loading branch information
AliAdnanSohail committed Sep 25, 2023
1 parent 9d997d6 commit 145a12e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion course_discovery/apps/course_metadata/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -3304,7 +3304,7 @@ def weeks_to_complete_max(self):
def marketing_url(self):
if self.marketing_slug:
if self.marketing_slug.find('/') != -1:
return self.marketing_slug
return urljoin(self.partner.marketing_site_url_root, self.marketing_slug)
path = f'{self.type.slug.lower()}/{self.marketing_slug}'
return urljoin(self.partner.marketing_site_url_root, path)

Expand Down
3 changes: 2 additions & 1 deletion course_discovery/apps/course_metadata/tests/test_models.py
Original file line number Diff line number Diff line change
Expand Up @@ -2763,7 +2763,8 @@ def test_marketing_url_without_slug(self):
def test_marketing_url__with_slashes(self):
""" Verify the property returns marketing url as it is if marketing_slug contains a slash"""
self.program.marketing_slug = 'type/subject/org-title'
assert self.program.marketing_url == self.program.marketing_slug
assert self.program.marketing_url == f"{self.program.partner.marketing_site_url_root}" \
f"{self.program.marketing_slug}"

def test_course_runs(self):
"""
Expand Down

0 comments on commit 145a12e

Please sign in to comment.