remove global staff check on enable_checklists_quality waffle flag
This commit is contained in:
committed by
Farhanah Sheets
parent
10dc80b205
commit
8da3278086
@@ -15,14 +15,13 @@ def should_show_checklists_page():
|
||||
return False
|
||||
|
||||
|
||||
def should_show_checklists_quality(requesting_user, course_key):
|
||||
def should_show_checklists_quality(course_key):
|
||||
"""
|
||||
Determine if the ENABLE_CHECKLISTS_QUALITY waffle flag is set
|
||||
and if the user is able to see it
|
||||
"""
|
||||
|
||||
if ENABLE_CHECKLISTS_QUALITY.is_enabled(course_key):
|
||||
if GlobalStaff().has_user(requesting_user):
|
||||
return True
|
||||
return True
|
||||
|
||||
return False
|
||||
|
||||
@@ -52,7 +52,7 @@
|
||||
"help_tokens": {
|
||||
"files": "${get_online_help_info(online_help_token())['doc_url'] | n, js_escaped_string}"
|
||||
},
|
||||
"enable_quality": ${should_show_checklists_quality(request.user, context_course.id) | n, dump_js_escaped_json},
|
||||
"enable_quality": ${should_show_checklists_quality(context_course.id) | n, dump_js_escaped_json},
|
||||
"links": {
|
||||
"certificates": ${reverse('certificates_list_handler', kwargs={'course_key_string': unicode(course_key)})| n, dump_js_escaped_json},
|
||||
"course_outline": ${reverse('course_handler', kwargs={'course_key_string': unicode(course_key)})| n, dump_js_escaped_json},
|
||||
|
||||
@@ -184,7 +184,7 @@ from django.core.urlresolvers import reverse
|
||||
"help_tokens": {
|
||||
"files": "${get_online_help_info(online_help_token())['doc_url'] | n, js_escaped_string}"
|
||||
},
|
||||
"enable_quality": ${should_show_checklists_quality(request.user, context_course.id) | n, dump_js_escaped_json},
|
||||
"enable_quality": ${should_show_checklists_quality(context_course.id) | n, dump_js_escaped_json},
|
||||
"links": {
|
||||
"settings": ${reverse('settings_handler', kwargs={'course_key_string': unicode(course_key)})| n, dump_js_escaped_json}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user