Remove course listing from user tab of sysadmin
I'm admittedly unclear why we decided to include a listing of _courses_ in the _users_ tab, when this listing is already available via the _courses_ tab, which is where you'd expect to find it. Particularly problematic here is the impact this has on page loads.. For instances with a large number of courses [1], pulling and rendering adds a nontrivial amount of overhead and leads to longer load times. In our case, this often leads to a page timeout. Since this is the main landing page for the sysadmin dashboard, it presents the appearance that the entire dashboard has crashed. In not being able to access the main page, you can't access any of the other tabs via navigation, unless you manually enter the URL :\ [1] ~750 courses, in one example
This commit is contained in:
@@ -183,15 +183,6 @@ class Users(SysadminDashboardView):
|
||||
self.datatable['data'] = [[_('Total number of users'),
|
||||
User.objects.all().count()]]
|
||||
|
||||
self.msg += HTML(u'<h2>{0}</h2>').format(
|
||||
_('Courses loaded in the modulestore')
|
||||
)
|
||||
self.msg += HTML(u'<ol>')
|
||||
for course in self.get_courses():
|
||||
self.msg += HTML(u'<li>{0} ({1})</li>').format(
|
||||
escape(text_type(course.id)), text_type(course.location))
|
||||
self.msg += HTML(u'</ol>')
|
||||
|
||||
def get(self, request):
|
||||
|
||||
if not request.user.is_staff:
|
||||
|
||||
Reference in New Issue
Block a user