Skip to content

Commit

Permalink
fix: adding logo/org name override (#776)
Browse files Browse the repository at this point in the history
  • Loading branch information
kiram15 authored Feb 15, 2024
1 parent 2cc2188 commit b2b95ea
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 2 deletions.
6 changes: 4 additions & 2 deletions enterprise_catalog/apps/catalog/algolia_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -475,13 +475,15 @@ def get_course_partners(course):
"""
partners = []
owners = course.get('owners') or []
org_name_override = course.get('organization_short_code_override')
logo_override = course.get('organization_logo_override_url')

for owner in owners:
partner_name = owner.get('name')
if partner_name:
partner_metadata = {
'name': partner_name,
'logo_image_url': owner.get('logo_image_url'),
'name': org_name_override or partner_name,
'logo_image_url': logo_override or owner.get('logo_image_url'),
}
partners.append(partner_metadata)

Expand Down
19 changes: 19 additions & 0 deletions enterprise_catalog/apps/catalog/tests/test_algolia_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -235,6 +235,25 @@ def test_get_course_card_image_url(self, course_metadata, expected_image_url):
},
],
),
(
{
'owners': [
{
'name': 'Test Org Name',
'logo_image_url': 'https://fake.image1',
'ignored_attr': None,
},
],
'organization_short_code_override': 'Org Name Override',
'organization_logo_override_url': 'https://fake.image2',
},
[
{
'name': 'Org Name Override',
'logo_image_url': 'https://fake.image2',
},
]
)
)
@ddt.unpack
def test_get_course_partners(self, course_metadata, expected_partners):
Expand Down

0 comments on commit b2b95ea

Please sign in to comment.