From c9b2bd8426f67c91c8028dd4dc8b3d2a98bc29ab Mon Sep 17 00:00:00 2001
From: Felipe <andres.bermudez@edunext.co>
Date: Mon, 13 Dec 2021 13:57:21 -0500
Subject: [PATCH] refactor: change the pattern for show the link that redirect
 to sku basket

---
 .../lms/templates/courseware/course_about.html  | 17 ++++++++++-------
 .../dashboard/_dashboard_course_listing.html    | 10 ++++++++--
 2 files changed, 18 insertions(+), 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
index f6eff30d..7c184d1a 100644
--- 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,11 @@ <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.
+                  ${Text(_("You must successfully complete {link_start}{prc_display}{link_end} before you begin this course.")).format(
+                    link_start=HTML('<a href="${ecommerce_payment_page}?sku=${student_not_enrollment_in_requirement}">'),
+                    link_end=HTML('</a>'),
+                    prc_display=course_requirements['courses'][0]['display'],
+                  )}
                 </p>
               </li>
             %else:
@@ -241,11 +244,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
index 9726d69e..6f0bcef9 100644
--- 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
@@ -499,9 +499,15 @@ <h4 class="message-title">${_('Your current verification will expire soon.')}</h
        % if course_requirements:
           % 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.
+              <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="${ecommerce_payment_page}?sku=${student_not_enrollment_in_requirement}">'),
+                    link_end=HTML('</a>'),
+                    prc_display=course_requirements['courses'][0]['display'],
+                  )}
+              </p>
             </div>
           %else:
             ## Multiple pre-requisite courses are not supported on frontend that's why we are pulling first element