Commit a31ae128 by Matt Drayer

Merge pull request #8808 from edx/rc/2015-07-08-mattdrayer

rc/2015-07-08-mattdrayer: Hide button behind feature flag
parents f9ce8264 ba7422da
...@@ -416,8 +416,9 @@ class CertificatesTest(BaseInstructorDashboardTest): ...@@ -416,8 +416,9 @@ class CertificatesTest(BaseInstructorDashboardTest):
""" """
Scenario: On the Certificates tab of the Instructor Dashboard, Generate Certificates button is visible. Scenario: On the Certificates tab of the Instructor Dashboard, Generate Certificates button is visible.
Given that I am on the Certificates tab on the Instructor Dashboard Given that I am on the Certificates tab on the Instructor Dashboard
Then I see 'Generate Certificates' button And the instructor-generation feature flag has been enabled
And when I click on 'Generate Certificates' button Then I see a 'Generate Certificates' button
And when I click on the 'Generate Certificates' button
Then I should see a status message and 'Generate Certificates' button should be disabled. Then I should see a status message and 'Generate Certificates' button should be disabled.
""" """
self.assertTrue(self.certificates_section.generate_certificates_button.visible) self.assertTrue(self.certificates_section.generate_certificates_button.visible)
......
...@@ -245,12 +245,15 @@ def _section_certificates(course): ...@@ -245,12 +245,15 @@ def _section_certificates(course):
for cert_status in example_cert_status for cert_status in example_cert_status
) )
) )
instructor_generation_enabled = settings.FEATURES.get('CERTIFICATES_INSTRUCTOR_GENERATION', False)
return { return {
'section_key': 'certificates', 'section_key': 'certificates',
'section_display_name': _('Certificates'), 'section_display_name': _('Certificates'),
'example_certificate_status': example_cert_status, 'example_certificate_status': example_cert_status,
'can_enable_for_course': can_enable_for_course, 'can_enable_for_course': can_enable_for_course,
'enabled_for_course': certs_api.cert_generation_enabled(course.id), 'enabled_for_course': certs_api.cert_generation_enabled(course.id),
'instructor_generation_enabled': instructor_generation_enabled,
'urls': { 'urls': {
'generate_example_certificates': reverse( 'generate_example_certificates': reverse(
'generate_example_certificates', 'generate_example_certificates',
......
...@@ -66,6 +66,7 @@ ...@@ -66,6 +66,7 @@
"AUTH_USE_OPENID_PROVIDER": true, "AUTH_USE_OPENID_PROVIDER": true,
"CERTIFICATES_ENABLED": true, "CERTIFICATES_ENABLED": true,
"CERTIFICATES_HTML_VIEW": true, "CERTIFICATES_HTML_VIEW": true,
"CERTIFICATES_INSTRUCTOR_GENERATION": true,
"MULTIPLE_ENROLLMENT_ROLES": true, "MULTIPLE_ENROLLMENT_ROLES": true,
"SOCIAL_SHARING_SETTINGS": { "SOCIAL_SHARING_SETTINGS": {
"CUSTOM_COURSE_URLS": true, "CUSTOM_COURSE_URLS": true,
......
...@@ -379,6 +379,9 @@ FEATURES = { ...@@ -379,6 +379,9 @@ FEATURES = {
# Certificates Web/HTML Views # Certificates Web/HTML Views
'CERTIFICATES_HTML_VIEW': False, 'CERTIFICATES_HTML_VIEW': False,
# Batch-Generated Certificates from Instructor Dashboard
'CERTIFICATES_INSTRUCTOR_GENERATION': False,
# Social Media Sharing on Student Dashboard # Social Media Sharing on Student Dashboard
'SOCIAL_SHARING_SETTINGS': { 'SOCIAL_SHARING_SETTINGS': {
# Note: Ensure 'CUSTOM_COURSE_URLS' has a matching value in cms/envs/common.py # Note: Ensure 'CUSTOM_COURSE_URLS' has a matching value in cms/envs/common.py
......
...@@ -56,6 +56,7 @@ ...@@ -56,6 +56,7 @@
% endif % endif
</div> </div>
% if section_data['instructor_generation_enabled']:
<hr /> <hr />
<div class="start-certificate-generation"> <div class="start-certificate-generation">
...@@ -78,4 +79,5 @@ ...@@ -78,4 +79,5 @@
<div class="no-pending-tasks-message"></div> <div class="no-pending-tasks-message"></div>
</div> </div>
%endif %endif
% endif
</div> </div>
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment