fixed import issue and added config value to hide assignments

edx_sga/management/commands/sga_migrate_submissions.py

@@ -7,9 +7,9 @@
 
 from django.core.management.base import BaseCommand, CommandError
 from lms.djangoapps.courseware.courses import get_course_by_id
-from lms.djangoapps.courseware.models import StudentModule
+from courseware.models import StudentModule
 from opaque_keys.edx.keys import CourseKey
-from common.djangoapps.student.models import anonymous_id_for_user
+from student.models import anonymous_id_for_user
 from submissions import api as submissions_api
 from xmodule.modulestore.django import modulestore
 

edx_sga/sga.py

@@ -25,9 +25,9 @@
 from django.utils.encoding import force_text
 from django.utils.timezone import now as django_now
 from django.utils.translation import ugettext as _
-from lms.djangoapps.courseware.models import StudentModule
+from courseware.models import StudentModule
 from safe_lxml import etree
-from common.djangoapps.student.models import user_by_anonymous_id
+from student.models import user_by_anonymous_id
 from submissions import api as submissions_api
 from submissions.models import StudentItem as SubmissionsStudent
 from submissions.models import Submission
@@ -39,6 +39,7 @@
 from xblockutils.studio_editable import StudioEditableXBlockMixin
 from xmodule.contentstore.content import StaticContent
 from xmodule.util.duedate import get_extended_due_date
+from openedx.core.djangoapps.site_configuration import helpers as configuration_helpers
 
 from edx_sga.constants import ITEM_TYPE
 from edx_sga.showanswer import ShowAnswerXBlockMixin
@@ -192,7 +193,7 @@ def add_xml_to_node(self, node):
         if 'solution' in node.attrib:
             # Try outputting it as an XML element if we can
             solution = node.attrib['solution']
-            wrapped = f"<solution>{solution}</solution>"
+            wrapped = "<solution>{}</solution>".format(solution)
             try:
                 child = etree.fromstring(wrapped)
             except:  # pylint: disable=bare-except
@@ -584,7 +585,8 @@ def student_view(self, context=None):
             "student_state": json.dumps(self.student_state()),
             "id": self.location.name.replace('.', '_'),
             "max_file_size": self.student_upload_max_size(),
-            "support_email": settings.TECH_SUPPORT_EMAIL
+            "support_email": settings.TECH_SUPPORT_EMAIL,
+            "enable_hide_assignment": configuration_helpers.get_value_theme('ENABLE_HIDE_ASSIGNMENT', False)
         }
         if self.show_staff_grading_interface():
             context['is_course_staff'] = True
@@ -792,6 +794,7 @@ def student_state(self):
             "upload_allowed": self.upload_allowed(submission_data=submission),
             "solution": solution,
             "base_asset_url": StaticContent.get_base_url_path_for_course_assets(self.location.course_key),
+            "enable_hide_assignment": configuration_helpers.get_value_theme('ENABLE_HIDE_ASSIGNMENT', False)
         }
 
     def staff_grading_data(self):

edx_sga/tasks.py

@@ -9,7 +9,7 @@
 from django.core.files.storage import default_storage
 from celery import shared_task
 from opaque_keys.edx.locator import BlockUsageLocator
-from common.djangoapps.student.models import user_by_anonymous_id
+from student.models import user_by_anonymous_id
 from submissions import api as submissions_api
 
 from edx_sga.constants import ITEM_TYPE

edx_sga/templates/staff_graded_assignment/show.html

@@ -14,6 +14,8 @@
         <label>
           <% if (uploaded) { %>
             {% trans "Upload a different file" %}
+          <% } else if (enable_hide_assignment) { %>
+            {% trans "Upload" %}
           <% } else { %>
             {% trans "Upload your assignment" %}
           <% } %>
@@ -38,7 +40,11 @@
               <a href="<%= annotatedUrl %>"><%= annotated.filename %></a><br/>
             <% } %>
           <% } else if (uploaded) { %>
-            {% trans "This assignment has not yet been graded." %}
+            <% if (enable_hide_assignment) { %>
+              {% trans "" %}
+            <% } else {%>
+              {% trans "This assignment has not yet been graded." %}
+            <% } %>
           <% } %>
         </p>
         <% if (upload_allowed) { %>
@@ -146,7 +152,7 @@
 
   <section aria-hidden="true" class="modal staff-modal" id="grade-{{ id }}" style="height: 75%" tabindex="-1">
     <div class="inner-wrapper" style="color: black; overflow: auto;">
-      <header><h2><span class="display_name">{{ display_name }}</span> - {% trans "Staff Graded Assignment" %}</h2></header>
+      <header><h2><span class="display_name">{{ display_name }}</span> - {% if enable_hide_assignment %} {% trans "Staff Graded" %} {% else %} {% trans "Staff Graded Assignment" %} {% endif %}</h2></header>
       <br/>
       <div>
         <a class="instructor-info-action button btn-download-all" href="#" id="download-init-button">{% trans "Download All Submissions" %}</a>

edx_sga/tests/integration_tests.py

@@ -20,11 +20,11 @@
 from django.db import transaction
 from django.test.utils import override_settings
 from lms.djangoapps.courseware import module_render as render
-from lms.djangoapps.courseware.models import StudentModule
+from courseware.models import StudentModule
 from lms.djangoapps.courseware.tests.factories import StaffFactory
 from opaque_keys.edx.locations import Location
 from opaque_keys.edx.locator import CourseLocator
-from common.djangoapps.student.models import UserProfile, anonymous_id_for_user
+from student.models import UserProfile, anonymous_id_for_user
 from common.djangoapps.student.tests.factories import AdminFactory
 from submissions import api as submissions_api
 from submissions.models import StudentItem