Moving 'enabled_reports' to Django setting; updating settings/base.py

Author: Jesse Shapiro

analytics_data_api/v0/views/courses.py

@@ -799,9 +799,7 @@ class ReportDownloadView(APIView):
             * expiration_date: The date through which the link will be valid
             * file_size: The size in bytes of the CSV download
     """
-    enabled_reports = (
-        'problem_response'
-    )
+    enabled_reports = settings.ENABLED_REPORT_IDENTIFIERS
 
     def get(self, _request, course_id, report_name):
         if report_name in self.enabled_reports:

analyticsdataserver/settings/base.py

@@ -85,15 +85,6 @@
 ########## END GENERAL CONFIGURATION
 
 
-########## MEDIA CONFIGURATION
-# See: https://docs.djangoproject.com/en/dev/ref/settings/#media-root
-MEDIA_ROOT = normpath(join(SITE_ROOT, 'media'))
-
-# See: https://docs.djangoproject.com/en/dev/ref/settings/#media-url
-MEDIA_URL = '/media/'
-########## END MEDIA CONFIGURATION
-
-
 ########## STATIC FILE CONFIGURATION
 # See: https://docs.djangoproject.com/en/dev/ref/settings/#static-root
 STATIC_ROOT = normpath(join(SITE_ROOT, 'assets'))
@@ -309,11 +300,12 @@
 # base url to generate link to user api
 LMS_USER_ACCOUNT_BASE_URL = None
 
-# storage settings for report downloads
+# settings for report downloads
 DEFAULT_FILE_STORAGE = 'django.core.files.storage.FileSystemStorage'
-MEDIA_ROOT = '/edx/app/analytics_api/analytics_api/static/reports'
+MEDIA_ROOT = normpath(join(SITE_ROOT, 'static', 'reports'))
 MEDIA_URL = 'http://localhost:8100/static/reports/'
 COURSE_REPORT_FILE_LOCATION_TEMPLATE = '{course_id}_{report_name}.csv'
+ENABLED_REPORT_IDENTIFIERS = ('problem_response',)
 
 ########## END ANALYTICS DATA API CONFIGURATION