refactor: fix behavior to redirect to sku basket in the course about …

…and dashboard
Pearson-Advance · Dec 16, 2021 · 8bbe3fb · 8bbe3fb
1 parent c048c0f
commit 8bbe3fb
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/edx-platform/pearson-pathways-theme/lms/templates/courseware/course_about.html b/edx-platform/pearson-pathways-theme/lms/templates/courseware/course_about.html
@@ -229,8 +229,8 @@ <h1>
                   </a>
                 </span>
                 <p class="tip">
-                  You must successfully complete <a href="${ecommerce_payment_page}?sku=${student_not_enrollment_in_requirement}">${Text(_("{prc_display}")).format(
-                  prc_display=course_requirements['display'])}</a> before you begin this course.
+                  You must successfully complete <a href="${ecommerce_payment_page}?sku=${student_not_enrollment_in_requirement}">
+                  ${Text(_("{prc_display}")).format(prc_display=course_requirements['display'])}</a> before you begin this course.
                 </p>
               </li>
             %else:
@@ -241,11 +241,11 @@ <h1>
                 ## Multiple pre-requisite courses are not supported on frontend that's why we are pulling first element
                 <span class="important-dates-item-text pre-requisite"><a href="${prc_target}">${course_requirements['display']}</a></span>
                 <p class="tip">
-                ${Text(_("You must successfully complete {link_start}{prc_display}{link_end} before you begin this course.")).format(
-                  link_start=HTML('<a href="{}">').format(prc_target),
-                  link_end=HTML('</a>'),
-                  prc_display=course_requirements['display'],
-                )}
+                  ${Text(_("You must successfully complete {link_start}{prc_display}{link_end} before you begin this course.")).format(
+                    link_start=HTML('<a href="{}">').format(prc_target),
+                    link_end=HTML('</a>'),
+                    prc_display=course_requirements['display'],
+                  )}
                 </p>
               </li>
             %endif

diff --git a/edx-platform/pearson-pathways-theme/lms/templates/dashboard/_dashboard_course_listing.html b/edx-platform/pearson-pathways-theme/lms/templates/dashboard/_dashboard_course_listing.html
@@ -500,8 +500,8 @@ <h4 class="message-title">${_('Your current verification will expire soon.')}</h
           % if student_not_enrollment_in_requirement:
             ## Multiple pre-requisite courses are not supported on frontend that's why we are pulling first element
             <div class="prerequisites">
-                You must successfully complete <a href="${ecommerce_payment_page}?sku=${student_not_enrollment_in_requirement}">${Text(_("{prc_display}")).format(
-                prc_display=course_requirements['courses'][0]['display'])}</a> before you begin this course.
+              You must successfully complete <a href="${ecommerce_payment_page}?sku=${student_not_enrollment_in_requirement}">
+              ${Text(_("{prc_display}")).format(prc_display=course_requirements['courses'][0]['display'],)}</a> before you begin this course.
             </div>
           %else:
             ## Multiple pre-requisite courses are not supported on frontend that's why we are pulling first element