also even when we have a marketing site, we still need a course image for the dashboard
This commit is contained in:
@@ -128,7 +128,10 @@ class CourseDetailsTestCase(CourseTestCase):
|
||||
self.assertContains(response, "Enrollment End Date")
|
||||
self.assertContains(response, "not the dates shown on your course summary page")
|
||||
|
||||
self.assertNotContains(response, "Introducing Your Course")
|
||||
self.assertContains(response, "Introducing Your Course")
|
||||
self.assertContains(response, "Course Image")
|
||||
self.assertNotContains(response,"Course Overview")
|
||||
self.assertNotContains(response,"Course Introduction Video")
|
||||
self.assertNotContains(response, "Requirements")
|
||||
|
||||
def test_regular_site_fetch(self):
|
||||
|
||||
@@ -184,8 +184,11 @@ CMS.Views.Settings.Details = CMS.Views.ValidatingView.extend({
|
||||
if (forcedTarget) {
|
||||
thisTarget = forcedTarget;
|
||||
thisTarget.id = $(thisTarget).attr('id');
|
||||
} else {
|
||||
} else if (e !== null) {
|
||||
thisTarget = e.currentTarget;
|
||||
} else
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
if (!this.codeMirrors[thisTarget.id]) {
|
||||
|
||||
@@ -194,14 +194,13 @@ from contentstore import utils
|
||||
% endif
|
||||
</section>
|
||||
<hr class="divide" />
|
||||
% if about_page_editable:
|
||||
<section class="group-settings marketing">
|
||||
<header>
|
||||
<h2 class="title-2">${_("Introducing Your Course")}</h2>
|
||||
<span class="tip">${_("Information for prospective students")}</span>
|
||||
</header>
|
||||
|
||||
<ol class="list-input">
|
||||
% if about_page_editable:
|
||||
<li class="field text" id="field-course-overview">
|
||||
<label for="course-overview">${_("Course Overview")}</label>
|
||||
<textarea class="tinymce text-editor" id="course-overview"></textarea>
|
||||
@@ -213,6 +212,7 @@ from contentstore import utils
|
||||
%>${text}</%def>
|
||||
<span class="tip tip-stacked">${overview_text()}</span>
|
||||
</li>
|
||||
% endif
|
||||
|
||||
<li class="field image" id="field-course-image">
|
||||
<label>${_("Course Image")}</label>
|
||||
@@ -242,6 +242,7 @@ from contentstore import utils
|
||||
</div>
|
||||
</li>
|
||||
|
||||
% if about_page_editable:
|
||||
<li class="field video" id="field-course-introduction-video">
|
||||
<label for="course-overview">${_("Course Introduction Video")}</label>
|
||||
<div class="input input-existing">
|
||||
@@ -258,9 +259,11 @@ from contentstore import utils
|
||||
<span class="tip tip-stacked">${_("Enter your YouTube video's ID (along with any restriction parameters)")}</span>
|
||||
</div>
|
||||
</li>
|
||||
% endif
|
||||
</ol>
|
||||
</section>
|
||||
|
||||
% if about_page_editable:
|
||||
<hr class="divide" />
|
||||
|
||||
<section class="group-settings requirements">
|
||||
@@ -277,7 +280,7 @@ from contentstore import utils
|
||||
</li>
|
||||
</ol>
|
||||
</section>
|
||||
% endif
|
||||
% endif
|
||||
</form>
|
||||
</article>
|
||||
<aside class="content-supplementary" role="complimentary">
|
||||
|
||||
Reference in New Issue
Block a user