Merge pull request #30227 from openedx/sajjad/VAN-921
fix: Make course metadata values dynamic
This commit is contained in:
@@ -10,6 +10,8 @@ from eventtracking import tracker
|
||||
|
||||
from openedx.core.djangoapps.site_configuration import helpers as configuration_helpers
|
||||
|
||||
from common.djangoapps.course_modes.models import CourseMode
|
||||
|
||||
log = logging.getLogger(__name__)
|
||||
|
||||
USER_SAVE_FOR_LATER_EMAIL_SENT = 'edx.bi.user.saveforlater.email.sent'
|
||||
@@ -26,6 +28,13 @@ def _get_program_pacing(course_runs):
|
||||
return 'Self-paced' if pacing == 'self_paced' else 'Instructor-led'
|
||||
|
||||
|
||||
def _get_course_price(course):
|
||||
"""
|
||||
Get price of a course
|
||||
"""
|
||||
return CourseMode.min_course_price_for_currency(course_id=str(course.id), currency='USD')
|
||||
|
||||
|
||||
def _get_event_properties(data):
|
||||
"""
|
||||
set event properties for course and program which are required in braze email template
|
||||
@@ -38,6 +47,7 @@ def _get_event_properties(data):
|
||||
|
||||
if data.get('type') == 'course':
|
||||
course = data.get('course')
|
||||
price = _get_course_price(course)
|
||||
event_properties.update({
|
||||
'properties': {
|
||||
'course_image_url': '{base_url}{image_path}'.format(
|
||||
@@ -55,6 +65,7 @@ def _get_event_properties(data):
|
||||
'max_effort': data.get('max_effort'),
|
||||
'pacing_type': 'Self-paced' if course.self_paced else 'Instructor-led',
|
||||
'type': 'course',
|
||||
'price': 'Free' if price == 0 else f'${price} USD',
|
||||
}
|
||||
})
|
||||
|
||||
|
||||
Reference in New Issue
Block a user