Fix errors in user-facing strings.

openassessment/templates/openassessmentblock/edit/oa_edit.html

@@ -113,13 +113,13 @@
                 </li>
                 <li id="openassessment_submission_latex_wrapper" class="field comp-setting-entry">
                     <div class="wrapper-comp-setting">
-                        <label for="openassessment_submission_latex_editor" class="setting-label">{% trans "Allow Latex Responses"%}</label>
+                        <label for="openassessment_submission_latex_editor" class="setting-label">{% trans "Allow LaTeX Responses"%}</label>
                         <select id="openassessment_submission_latex_editor" class="input setting-input" name="latex submission">
                             <option value="0">{% trans "False"%}</option>
                             <option value="1" {% if allow_latex %} selected="true" {% endif %}>{% trans "True"%}</option>
                         </select>
                     </div>
-                    <p class="setting-help">{% trans "Specify whether learners can write Latex formatted strings"%}</p>
+                    <p class="setting-help">{% trans "Specify whether learners can write LaTeX formatted strings"%}</p>
                 </li>
                 <li id="openassessment_leaderboard_wrapper" class="field comp-setting-entry">
                     <div class="wrapper-comp-setting">
@@ -139,7 +139,7 @@
 
             <p class="openassessment_description" id="openassessment_step_select_description">
                 {% if 'example_based_assessment' in editor_assessments_order %}
-                    {% trans "In this assignment, you can include steps for learner training, peer assessment, self assessment, and example based assessment. Select the steps that you want below, and then drag them into the order that you want. If you include an example based assessment step, it must precede all other steps. If you include a learner training training step, it must precede peer and self assessment steps." %}
+                    {% trans "In this assignment, you can include steps for learner training, peer assessment, self assessment, and example based assessment. Select the steps that you want below, and then drag them into the order that you want. If you include an example based assessment step, it must precede all other steps. If you include a learner training step, it must precede peer and self assessment steps." %}
                 {% else %}
                     {% trans "In this assignment, you can include steps for learner training, peer assessment, and self assessment. Select the steps that you want below, and then drag them into the order that you want. If you include a learner training step, it must precede all other steps." %}
                 {% endif %}

openassessment/templates/openassessmentblock/edit/oa_edit_rubric.html

@@ -38,7 +38,7 @@
                     <textarea id="openassessment_rubric_feedback" class="input setting-input">{{ feedbackprompt }}</textarea>
                 </div>
                 <p class="setting-help">
-                    {% trans "Encourage your learners to provide feedback on the response they've graded. You can replace the sample text with your own." %}
+                    {% trans "Encourage your learners to provide feedback on the response they have graded. You can replace the sample text with your own." %}
                 </p>
             </li>
             <li class="field comp-setting-entry">

openassessment/templates/openassessmentblock/staff_debug/student_info.html

@@ -236,6 +236,6 @@
         {% endif %}
     </div>
     {% else %}
-        {% trans "Couldn't find a response for this learner." %}
+        {% trans "A response was not found for this learner." %}
     {% endif %}
 </div>

openassessment/templates/openassessmentblock/student_training/student_training_error.html

@@ -9,7 +9,7 @@
                 <h3 class="message__title">{% trans "Error Loading Learner Training Examples" %}</h3>
 
                 <div class="message__content">
-                    <p>{% trans "We couldn't load the student training step of this assignment." %}</p>
+                    <p>{% trans "The Learner Training Step of this assignment could not be loaded." %}</p>
                 </div>
             </div>
         </div>

openassessment/xblock/defaults.py

@@ -75,7 +75,7 @@ I noticed that this response...
 DEFAULT_EXAMPLE_ANSWER = (
     "Replace this text with your own sample response for this assignment. "
     "Then, under Response Score to the right, select an option for each criterion. "
-    "Learners learn to assess responses by assessing this response and comparing "
+    "Learners practice performing peer assessments by assessing this response and comparing "
     "the options that they select in the rubric with the options that you specified."
 )
 

openassessment/xblock/staff_info_mixin.py

@@ -376,10 +376,12 @@ class StaffInfoMixin(object):
                 cancelled_by_id=student_item_dict['student_id'],
                 assessment_requirements=assessment_requirements
             )
-            return {"success": True, 'msg': self._(u"The learner submission has been removed from peer assessment. "
-                                                   u"The learner receives a grade of zero unless you reset "
-                                                   u"the learner's attempts for the problem to allow them to "
-                                                   u"resubmit a response.")}
+            return {"success": True, 'msg': self._(
+                u"The learner submission has been removed from peer assessment. "
+                u"The learner receives a grade of zero unless you reset "
+                u"the learner's attempts for the problem to allow them to "
+                u"resubmit a response."
+            )}
         except (
                 AssessmentWorkflowError,
                 AssessmentWorkflowInternalError

openassessment/xblock/static/js/src/studio/oa_edit_listeners.js

@@ -95,7 +95,7 @@ OpenAssessment.StudentTrainingListener.prototype = {
         if (criterionAdded) {
             this.displayAlertMsg(
                 gettext("Criterion Added"),
-                gettext("You've added a criterion. You'll need to select an option for the criterion in the Learner Training step. To do this, click the Settings tab.")
+                gettext("You have added a criterion. You will need to select an option for the criterion in the Learner Training step. To do this, click the Settings tab.")
             );
         }
     },
@@ -142,7 +142,7 @@ OpenAssessment.StudentTrainingListener.prototype = {
         if (invalidated) {
             this.displayAlertMsg(
                 gettext("Option Deleted"),
-                gettext("You've deleted an option. The system has removed that option from its criterion in the sample responses in the Learner Training step. You may have to select a new option for the criterion.")
+                gettext("You have deleted an option. That option has been removed from its criterion in the sample responses in the Learner Training step. You might have to select a new option for the criterion.")
             );
         }
     },
@@ -174,7 +174,7 @@ OpenAssessment.StudentTrainingListener.prototype = {
         if (changed) {
             this.displayAlertMsg(
                 gettext("Option Deleted"),
-                gettext("You've deleted all the options for this criterion. The system has removed the criterion from the sample responses in the Learner Training step.")
+                gettext("You have deleted all the options for this criterion. The criterion has been removed from the sample responses in the Learner Training step.")
             );
         }
     },
@@ -201,7 +201,7 @@ OpenAssessment.StudentTrainingListener.prototype = {
         if (changed) {
             this.displayAlertMsg(
                 gettext("Criterion Deleted"),
-                gettext("You've deleted a criterion. The system has removed the criterion from the sample responses in the Learner Training step.")
+                gettext("You have deleted a criterion. The criterion has been removed from the example responses in the Learner Training step.")
             );
         }
     },

openassessment/xblock/student_training_mixin.py

@@ -49,8 +49,8 @@ class StudentTrainingMixin(object):
 
         try:
             path, context = self.training_path_and_context()
-        except: # pylint:disable=W0702
-            msg = u"Could not render learner training step for submission {}".format(self.submission_uuid)
+        except:  # pylint:disable=W0702
+            msg = u"Could not render Learner Training step for submission {}.".format(self.submission_uuid)
             logger.exception(msg)
             return self.render_error(self._(u"An unexpected error occurred."))
         else:
@@ -140,7 +140,7 @@ class StudentTrainingMixin(object):
         return template, context
 
     @XBlock.json_handler
-    def training_assess(self, data, suffix=''): # pylint:disable=W0613
+    def training_assess(self, data, suffix=''):  # pylint:disable=W0613
         """
         Compare the scores given by the student with those given by the course author.
         If they match, update the training workflow.  The client can then reload this
@@ -185,8 +185,7 @@ class StudentTrainingMixin(object):
             )
         except student_training.StudentTrainingRequestError:
             msg = (
-                u"Could not check learner training scores for "
-                u"the learner with submission UUID {uuid}"
+                u"Could not check learner training scores for the learner with submission UUID {uuid}"
             ).format(uuid=self.submission_uuid)
             logger.warning(msg, exc_info=True)
             return {

openassessment/xblock/test/test_staff_info.py

@@ -102,7 +102,7 @@ class TestCourseStaff(XBlockHandlerTestCase):
         # If we ARE course staff, then we should see the debug info
         xblock.xmodule_runtime.user_is_staff = True
         resp = self.request(xblock, 'render_student_info', json.dumps({}))
-        self.assertIn("couldn\'t find a response for this learner.", resp.decode('utf-8').lower())
+        self.assertIn("a response was not found for this learner.", resp.decode('utf-8').lower())
 
     @scenario('data/basic_scenario.xml')
     def test_hide_course_staff_debug_info_in_studio_preview(self, xblock):
@@ -169,7 +169,7 @@ class TestCourseStaff(XBlockHandlerTestCase):
         request.params = {"student_id": "test_student"}
         # Verify that we can render without error
         resp = xblock.render_student_info(request)
-        self.assertIn("couldn\'t find a response for this learner.", resp.body.lower())
+        self.assertIn("a response was not found for this learner.", resp.body.lower())
 
     @scenario('data/peer_only_scenario.xml', user_id='Bob')
     def test_staff_debug_student_info_peer_only(self, xblock):

openassessment/xblock/validation.py

@@ -293,7 +293,7 @@ def validate_assessment_examples(rubric_dict, assessments, _):
 
             # Must have at least one training example
             if len(examples) == 0:
-                return False, _(u"Learner training and example-based assessments must have at least one training example")
+                return False, _(u"Learner training and example-based assessments must have at least one training example.")
 
             # Delegate to the student training API to validate the
             # examples against the rubric.