Move Pearson URL to settings. Fix Register button. Other small changes.

common/djangoapps/student/models.py

@@ -499,6 +499,10 @@ class TestCenterRegistration(models.Model):
     def get_accommodation_names(self):
         return [ ACCOMMODATION_CODE_DICT.get(code, "Unknown code " + code) for code in self.get_accommodation_codes() ]         
 
+    @property
+    def registration_signup_url(self):
+        return settings.PEARSONVUE_SIGNINPAGE_URL
+    
 class TestCenterRegistrationForm(ModelForm):
     class Meta:
         model = TestCenterRegistration

lms/envs/common.py

@@ -328,6 +328,9 @@ STAFF_GRADING_INTERFACE = None
 # Used for testing, debugging
 MOCK_STAFF_GRADING = False
 
+################################# Pearson TestCenter config  ################
+
+PEARSONVUE_SIGNINPAGE_URL = "https://www1.pearsonvue.com/testtaker/signin/SignInPage/EDX"
 
 ################################# Jasmine ###################################
 JASMINE_TEST_DIRECTORY = PROJECT_ROOT + '/static/coffee'

lms/templates/dashboard.html

@@ -236,14 +236,12 @@
                 % if registration is not None:
                     % if registration.is_accepted:  
                 <div class="message message-status is-shown exam-schedule">
-                	<!-- TODO: pull Pearson destination out into a Setting -->
-                   <a href="https://www1.pearsonvue.com/testtaker/signin/SignInPage/EDX" class="exam-button">Schedule Pearson exam</a>
+                   <a href="${registration.registration_signup_url}" class="button exam-button">Schedule Pearson exam</a>
                    <p class="exam-registration-number"><a href="${testcenter_register_target}" id="exam_register_link">Registration</a> number: <strong>${registration.client_authorization_id}</strong></p>
                    <p class="message-copy">Write this down! You’ll need it to schedule your exam.</p>
                 </div>
                     % endif
                     % if  registration.is_rejected:	    
-                	<!-- TODO: revise rejection text -->
                 <div class="message message-status is-shown exam-schedule">
                   <a href="mailto:exam-help@edx.org?subject=Pearson VUE Exam - ${get_course_about_section(course, 'university')} ${course.number}" class="button contact-button">Contact exam-help@edx.org</a>
                   <p class="message-copy">Your 

lms/templates/test_center_register.html

@@ -46,7 +46,7 @@
         $(".submission-error").addClass("is-shown");
         for (fieldname in field_errors) {
           // to inform a user of what field the error occurred on, add a class of .error to the .field element.
-          // for convenience, use the "data-field" attribute to identify the one matching the errant field's name.
+          // for convenience, use the "id" attribute to identify the one matching the errant field's name.
           var field_id = "field-" + fieldname;
           var field_label = $("[id='"+field_id+"'] label").text();
 
@@ -117,7 +117,7 @@
   <section class="status message message-flash registration-processed message-action">
     <h3 class="message-title">Your registration for the Pearson exam has been processed</h3>
     <p class="message-copy">Your registration number is <strong>${registration.client_authorization_id}</strong> (Write this down! You’ll need it to schedule your exam.)</p>
-    <a href="https://www1.pearsonvue.com/testtaker/signin/SignInPage/EDX" class="button exam-button">Schedule Pearson exam</a>
+    <a href="${registration.registration_signup_url}" class="button exam-button">Schedule Pearson exam</a>
   </section>
    % endif
 
@@ -138,7 +138,7 @@
       
       % if registration:
       <p class="instructions">
-        Please complete the following form to update your demographic information used in your Pearson VUE Proctored Exam. Required fields are noted by <strong class="indicator">bold text and an asterisk (*)</strong>. 
+        Please use the following form if you need to update your demographic information used in your Pearson VUE Proctored Exam. Required fields are noted by <strong class="indicator">bold text and an asterisk (*)</strong>. 
       </p>
       % else: 
       <p class="instructions">
@@ -264,7 +264,9 @@
       % endif
 
         % if registration:
+          % if registration.accommodation_request and len(registration.accommodation_request) > 0:
         <p class="note"><span class="title">Note</span>: Accomodation requests, which need to be reviewed in detail, <strong>will add significant delay to the registration process</strong>.</p>
+          % endif
         % else: 
         <p class="note"><span class="title">Note</span>: Accommodation requests are not part of your demographic information, <strong>and cannot be changed once submitted</strong>. Accomodation requests, which need to be reviewed in detail, <strong>will add significant delay to the registration process</strong>.</p>
         % endif
@@ -275,13 +277,13 @@
           <ol class="list-input">
             % if registration:
               % if registration.accommodation_request and len(registration.accommodation_request) > 0:
-            <li data-field="accommodation_request" class="field submitted" id="field-accommodation_request">
+            <li class="field submitted" id="field-accommodation_request">
               <label for="accommodation_request">Accommodations Requested</label>
               <p class="value" id="accommodations">${registration.accommodation_request}</p>
             </li>
               % endif
             % else: 
-            <li data-field="accommodation_request" class="field" id="field-accommodation_request">
+            <li class="field" id="field-accommodation_request">
               <label for="accommodation_request">Accommodations Requested</label>
               <textarea class="long" id="accommodation_request" name="accommodation_request" value="" placeholder=""></textarea>             
             </li>