Merge pull request #5290 from mitocw/bdero/fix-grade-summary-opaque-keys

Remove unimplemented legacy dashboard grade_summary feature

lms/djangoapps/instructor/views/legacy.py

@@ -1427,18 +1427,6 @@ def get_student_grade_summary_data(request, course, get_grades=True, get_raw_sco
 
 # Gradebook has moved to instructor.api.spoc_gradebook #
 
-
-@cache_control(no_cache=True, no_store=True, must_revalidate=True)
-def grade_summary(request, course_key):
-    """Display the grade summary for a course."""
-    course = get_course_with_access(request.user, 'staff', course_key)
-
-    # For now, just a page
-    context = {'course': course,
-               'staff_access': True, }
-    return render_to_response('courseware/grade_summary.html', context)
-
-
 #-----------------------------------------------------------------------------
 # enrollment
 

lms/templates/courseware/grade_summary.html

-<%! from django.utils.translation import ugettext as _ %>
-<%inherit file="/main.html" />
-<%! from django.core.urlresolvers import reverse %>
-<%namespace name='static' file='/static_content.html'/>
-
-<%include file="/courseware/course_navigation.html" args="active_page=''" />
-
-<section class="container">
-<div class="gradebook-summary-wrapper">
-  <section class="gradebook-summary-content">
-    <h1>${_("Grade summary")}</h1>
-
-    <p>${_("Not implemented yet")}</p>
-
-  </section>
-</div>
-</section>

lms/templates/courseware/instructor_dashboard.html

@@ -208,10 +208,6 @@ function goto( mode)
     </p>
 
     <p>
-    <a href="${reverse('grade_summary_legacy', kwargs=dict(course_id=course.id.to_deprecated_string()))}" class="${'is-disabled' if disable_buttons else ''}">${_("Grade summary")}</a>
-    </p>
-
-    <p>
     <input type="submit" name="action" value="Dump list of enrolled students" class="${'is-disabled' if disable_buttons else ''}">
     </p>
 

lms/urls.py

@@ -410,8 +410,6 @@ if settings.COURSEWARE_ENABLED:
 
 if settings.COURSEWARE_ENABLED and settings.FEATURES.get('ENABLE_INSTRUCTOR_LEGACY_DASHBOARD'):
     urlpatterns += (
-        url(r'^courses/{}/legacy_grade_summary$'.format(settings.COURSE_ID_PATTERN),
-            'instructor.views.legacy.grade_summary', name='grade_summary_legacy'),
         url(r'^courses/{}/legacy_instructor_dash$'.format(settings.COURSE_ID_PATTERN),
             'instructor.views.legacy.instructor_dashboard', name="instructor_dashboard_legacy"),
     )