mattdrayer/fix-progress/template: Improved web certificates support

lms/djangoapps/courseware/tests/test_views.py

@@ -858,7 +858,8 @@ class ProgressPageTests(ModuleStoreTestCase):
 
         resp = views.progress(self.request, course_id=unicode(self.course.id))
         self.assertContains(resp, u"View Certificate")
-        self.assertContains(resp, u"You can now view your certificate")
+
+        self.assertContains(resp, u"You can now access your certificate")
         cert_url = certs_api.get_certificate_url(
             user_id=self.user.id,
             course_id=self.course.id,

lms/templates/courseware/progress.html

@@ -51,28 +51,24 @@ from django.utils.http import urlquote_plus
       %if show_generate_cert_btn:
       <div class="wrapper-msg wrapper-auto-cert">
         <div id="errors-info" class="errors-info"></div>
-
         %if passed:
         <div class="auto-cert-message" id="course-success">
           <div class="has-actions">
-                        % if is_downloadable and download_url:
-
+              <% post_url = reverse('generate_user_cert', args=[unicode(course.id)]) %>
+              % if is_downloadable:
               <div class="msg-content">
                 <h2 class="title">${_("Your certificate is available")}</h2>
-                            %if show_cert_web_view:
-                            <p class="copy">${_("You can now view your certificate.")}</p>
-                            %else:
-                            <p class="copy">${_(
-                                "You can now download your certificate as a PDF. If you keep working and receive a higher grade, you can request an updated certificate.")}
+                <p class="copy">
+                  ${_("You can now access your certificate. If you keep working and receive a higher grade,you can request an {link_start} updated certificate {link_end}.").format(
+                    link_start=u"<a class='generate_certs' href='#' data-endpoint={}>".format(post_url), link_end=u"</a>")}
                 </p>
-                            %endif
               </div>
               <div class="msg-actions">
-                            %if show_cert_web_view:
+                %if show_cert_web_view and cert_web_view_url:
                 <a class="btn" href="${cert_web_view_url}" target="_blank" title="${_('View certificate in a new browser window or tab.')}">
                   ${_("View Certificate")}
                 </a>
-                            %else:
+                %elif download_url:
                 <a class="btn" href="${download_url}" target="_blank" title="${_('PDF will open in a new browser window or tab.')}">
                   ${_("Download Your Certificate")}
                 </a>
@@ -90,7 +86,7 @@ from django.utils.http import urlquote_plus
                 <p class="copy">${_("You can keep working for a higher grade, or request your certificate now.")}</p>
               </div>
               <div class="msg-actions">
-                            <button class="btn generate_certs" data-endpoint="${reverse('generate_user_cert', args=[unicode(course.id)])}" id="btn_generate_cert">${_('Request Certificate')}</button>
+                <button class="btn generate_certs" data-endpoint="${post_url}" id="btn_generate_cert">${_('Request Certificate')}</button>
               </div>
               %endif
           </div>