Merge pull request #19780 from edx/aehsan/LEARNER-6943/adding_logs_to_check_discovery_response
Adding logs
This commit is contained in:
@@ -660,11 +660,16 @@ def _get_custom_template_and_language(course_id, course_mode, course_language):
|
|||||||
template = get_certificate_template(course_id, course_mode, closest_released_language)
|
template = get_certificate_template(course_id, course_mode, closest_released_language)
|
||||||
|
|
||||||
if template and template.language:
|
if template and template.language:
|
||||||
|
log.info("Returning template for course: {course_id} and template language is {language}".format(
|
||||||
|
course_id=course_id, language=template.language))
|
||||||
return (template, closest_released_language)
|
return (template, closest_released_language)
|
||||||
elif template:
|
elif template:
|
||||||
user_language = translation.get_language()
|
user_language = translation.get_language()
|
||||||
|
log.info("Returning template for course: {course_id} and template language is returned from settings"
|
||||||
|
" {language}".format(course_id=course_id, language=user_language))
|
||||||
return (template, user_language)
|
return (template, user_language)
|
||||||
else:
|
else:
|
||||||
|
log.info("No template found for course: {course_id}" .format(course_id=course_id))
|
||||||
return (None, None)
|
return (None, None)
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -513,6 +513,7 @@ def get_course_run_details(course_run_key, fields):
|
|||||||
|
|
||||||
cache_key = '{base}.course_runs'.format(base=catalog_integration.CACHE_KEY)
|
cache_key = '{base}.course_runs'.format(base=catalog_integration.CACHE_KEY)
|
||||||
|
|
||||||
|
logger.info("getting course run detail for the course: {course_key}".format(course_key=course_run_key))
|
||||||
course_run_details = get_edx_api_data(catalog_integration, 'course_runs', api, resource_id=course_run_key,
|
course_run_details = get_edx_api_data(catalog_integration, 'course_runs', api, resource_id=course_run_key,
|
||||||
cache_key=cache_key, many=False, traverse_pagination=False, fields=fields)
|
cache_key=cache_key, many=False, traverse_pagination=False, fields=fields)
|
||||||
return course_run_details
|
return course_run_details
|
||||||
|
|||||||
@@ -55,13 +55,16 @@ def get_edx_api_data(api_config, resource, api, resource_id=None, querystring=No
|
|||||||
|
|
||||||
cached = cache.get(cache_key)
|
cached = cache.get(cache_key)
|
||||||
if cached:
|
if cached:
|
||||||
|
log.info("Cached course run was returned for the course: {resource_id}".format(resource_id=resource_id))
|
||||||
return zunpickle(cached)
|
return zunpickle(cached)
|
||||||
|
|
||||||
try:
|
try:
|
||||||
endpoint = getattr(api, resource)
|
endpoint = getattr(api, resource)
|
||||||
querystring = querystring if querystring else {}
|
querystring = querystring if querystring else {}
|
||||||
|
log.info("Hitting discovery for course run:{resource_id}".format(resource_id=resource_id))
|
||||||
response = endpoint(resource_id).get(**querystring)
|
response = endpoint(resource_id).get(**querystring)
|
||||||
|
log.info("Response from discovery: {response} for the course: {resource_id}".format(response=response,
|
||||||
|
resource_id=resource_id))
|
||||||
if resource_id is not None:
|
if resource_id is not None:
|
||||||
if fields:
|
if fields:
|
||||||
results = get_fields(fields, response)
|
results = get_fields(fields, response)
|
||||||
|
|||||||
Reference in New Issue
Block a user