Skip to content

Commit

Permalink
fix: relocate course organization and return removed prerequisites info
Browse files Browse the repository at this point in the history
  • Loading branch information
ihor-romaniuk committed Apr 18, 2024
1 parent 3d0f61e commit 53e9ea0
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions lms/templates/courseware/course_about.html
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,6 @@
<section class="intro">
<div class="heading-group">
<h1>${course.display_name_with_default}</h1>
<span>${course.display_org_with_default}</span>
<br />
<p>${get_course_about_section(request, course, 'short_description')}</p>
</div>
Expand Down Expand Up @@ -159,7 +158,16 @@ <h1>${course.display_name_with_default}</h1>

<%block name="course_about_important_dates">
<ol class="important-dates">
<li class="important-dates-item"><span class="icon fa fa-info-circle" aria-hidden="true"></span><p class="important-dates-item-title">${_("Course Number")}</p><span class="important-dates-item-text course-number">${course.display_number_with_default}</span></li>
<li class="important-dates-item">
<span class="icon fa fa-building" aria-hidden="true"></span>
<p class="important-dates-item-title">${_("Organization")}</p>
<span class="important-dates-item-text course-org">${course.display_org_with_default}</span>
</li>
<li class="important-dates-item">
<span class="icon fa fa-info-circle" aria-hidden="true"></span>
<p class="important-dates-item-title">${_("Course Number")}</p>
<span class="important-dates-item-text course-number">${course.display_number_with_default}</span>
</li>
% if not course.start_date_is_still_default:
<%
course_start_date = course.advertised_start or course.start
Expand Down Expand Up @@ -217,6 +225,8 @@ <h1>${course.display_name_with_default}</h1>
<li class="prerequisite-course important-dates-item">
<span class="icon fa fa-list-ul" aria-hidden="true"></span>
<p class="important-dates-item-title">${_("Prerequisites")}</p>
## 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}">${pre_requisite_courses[0]['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),
Expand Down

0 comments on commit 53e9ea0

Please sign in to comment.